Status
()
People
(Reporter: rods (gone), Assigned: anthonyd)
Tracking
Firefox Tracking Flags
(Not tracked)
Details
(Whiteboard: FIX IN HAND [nsbeta1+])
Attachments
(2 attachments)
|
748 bytes,
patch
|
Details | Diff | Splinter Review | |
|
3.85 KB,
patch
|
Details | Diff | Splinter Review |
Opening a file with this content: <form> <select> <option>One</option> <option>Two</option> <option>Three</option> </select> <form> Saves it out like this: <html> <head> </head> <body> <form> <select><option -moz-option-selected="">One</option><option>Two</option><option> Three</option></select></form> <form></form> </body> </html> The -moz-option-selected="" part should be written out.
|
||
Comment 1•18 years ago
|
||
reassign to anthonyd for mozilla .9 release. Akkana thinks this is probably pretty easy (typo using "-moz" rather than "_moz").
Assignee: beppe → anthonyd
Target Milestone: --- → mozilla0.9
|
||
Comment 2•18 years ago
|
||
nsHTMLContentSerializer filters out any attribute beginning with _moz. We need to find out why the select code is using -moz instead of _moz. (-moz is the de-facto standard for CSS, but _moz is the de-facto standard for attributes, if you do an lxr search; Simon and I checked that a while back when I was wondering whether we should convert all the editor's uses of _moz to -moz in attributes to be consistent with our CSS.)
|
||
Comment 4•18 years ago
|
||
adding keywords and whiteboard data
Keywords: correctness, nsbeta1
Whiteboard: [nsbeta1+]
Created attachment 24687 [details] [diff] [review] one line fix for this bug
fix is as easy as everyone said it would be. adding kin to the list. need a= from akkana and sr= from kin for mozilla 0.9 check in. anthonyd
Whiteboard: [nsbeta1+] → FIX IN HAND [nsbeta1+]
|
|
||
Comment 7•17 years ago
|
||
I'm fine with the fix, but have you run an lxr search to make sure no one else was depending on the old name? I'd recommend getting an r or sr from a layout person to make sure it's okay with them.
That fix isn't right ... it will change the usage in the CSS files too. -moz is a standard they use in layout for moz specific CSS properties.
kin and i talked with rods and he is using -moz-option-selected as an attribute, so renaming it to _moz-option-selected in the atom list and the appropos css files SHOULD be ok. will attach new diff with css changes. anthonyd
|
(Assignee) | |
Comment 10•17 years ago
|
||
Created attachment 24695 [details] [diff] [review] new diff with css changes and atom list change
|
||
Comment 11•17 years ago
|
||
This looks OK to me, [s]r=attinasi Has Rod reviewed it?
|
|
(Assignee) | |
Comment 12•17 years ago
|
||
adding rods to cc list. rod, can i get a r= please? thanks, anthonyd
|
|
(Assignee) | |
Comment 13•17 years ago
|
||
setting to 0.8.1. i would like to check this in, i have an sr=, but i need an r=. kin has reviewd it ans said its ok, but i guess i need a r= from layout. anthonyd
Target Milestone: mozilla0.9 → mozilla0.8.1
|
(Reporter) | |
Comment 14•17 years ago
|
||
r=rods
|
|
(Assignee) | |
Comment 15•17 years ago
|
||
checked in anthonyd
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
|
||
Comment 16•17 years ago
|
||
Rod, is this working for you now ? thanks...
|
|
||
Comment 17•17 years ago
|
||
Tony is gonna verify this one...thanks!
|
|
(Assignee) | |
Comment 18•17 years ago
|
||
re-opening bug, when jst split layout he didnt pick up my changes to fix this bug. some of my changes are still in the tree: /webshell/tests/viewer/samples/aform.css /webshell/tests/viewer/samples/bform.css /webshell/tests/viewer/samples/demoform.css /webshell/tests/viewer/samples/mozform.css but my changes to nsLayoutAtomlist.h and forms.css are gone. adding jst to the cc list.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
|
|
(Assignee) | |
Comment 19•17 years ago
|
||
ok, when the tree opens back up, i will re land the fix for this. anthonyd
|
|
(Assignee) | |
Comment 20•17 years ago
|
||
rechecked in fix. i also re-verified it in my build, but i will do it again with actual opt bits
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago → 17 years ago
Resolution: --- → FIXED
|
|
||
Comment 21•17 years ago
|
||
Tony, if you can verify with opt bits...thanks! also go ahead and mark this puppy VERIFIED-FIXED after doing so.. thanks dude.
|
|
||
Comment 22•17 years ago
|
||
marking verified...Tony checked it out in debug build.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•