Hi,
The yes_no widget doesn't work at all.
Please fix it. Or, scrap it (I don't think anybody is using it just yet!)
All of the other widgets seem to work fine.
Bye,
Merc.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | test.patch | 2.79 KB | mercmobily |
| #9 | test.css_.txt | 4.66 KB | mercmobily |
| #8 | extra_voting_forms.patch | 9.22 KB | cedricfontaine |
| #5 | yes_no.patch | 9.46 KB | cedricfontaine |
Comments
Comment #1
sikjoy commentedComment #2
gorillapictures commentedi am using it....isn't working...
Comment #3
fluxline commentedhow are you using it if it isn't working? just askin ...
Comment #4
mercmobily commentedHi,
It would be great to fix this. sicjoy, is this within your reach?
Merc.
Comment #5
cedricfontaine commentedSeems to be related to CSS. This patch should fixed it
Comment #6
mercmobily commentedHi,
A few notes about this patch:
1) I think you added loads of CR (carriage returns). Please watch out! Also, are you maintaining the same spacing?
2) Please make .diff files relative to the module's directory. In your case, the "patch" command looked for " modules/extra_voting_forms/widgets/yes_no/extra_voting_forms_all.css" -- however, you should create the patch while in "modules/extra_voting_forms", so that the patch command only uses the path " widgets/yes_no/extra_voting_forms_all.css" .
3) The patch fails! I am not sure why, but I can't apply this to CVS.
Please let me know :-D
Bye!
Merc.
Comment #7
Leeteq commentedSubscribing.
Comment #8
cedricfontaine commentedPlease confirm with this one
Comment #9
mercmobily commentedHi,
The patch still won't apply. Which command have you run?
I created a file called "test.css", where I put what I *thought* your patch should do. In fact, looking at your patch, it seems to delete pretty much everything and add everything from scratch, but starting from the wrong line number (?).
Then I ran " diff -up widgets/yes_no/extra_voting_forms_all.css widgets/yes_no/test.css > /tmp/test.patch".
The result is in attach. You will see test.css (which is what I *think* you wanted to do, please confirm) and test.patch.
I think it would be good to figure out why you were generating patches that wouldn't apply...!
Bye,
Merc.
Comment #10
cedricfontaine commentedYour test.css seems ok except the last line with a + sign at the beginning.
I don't know why but I have carriage returns problem with my working file, which breaks diff.
Also, I think we should remove widgets/yes_no/extra_voting_forms_js.css as it's not used.
Comment #11
mercmobily commentedHi,
OK, all good :-D
Big finally closedddddddddddddd!!!
** Thank you Cedric**
Merc.
Comment #12
mercmobily commented