Composer, in CSS mode, should be able to unbold or remove italic font style even if the markup is invalid, for instance if a B contains a P.
This bug helping the editing of html documents exported by Microsoft products with invalid markup, I am setting P2 098. Also setting nsDogFood keyword given the fact the Focal Review html form contains this kind of invalid markup. Setting EDITORBASE 1 day.
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: EDITORBASE 1 day
Target Milestone: --- → mozilla0.9.8
Created attachment 64485 [details] test case for this bug 1. launch composer in CSS mode 2. open test case 3. select "BBBBB" 4. click on the B button (resp. the I button) expected result : "BBBBB" gets normal font weight (resp. roman font style) 5. select "inside" 6. click on B button expected result : "inside" gets normal font weight
Created attachment 64486 [details] [diff] [review] patch v1.0 Patch for this bug; Joe and Kin, can you review please ?
Should the comment that says: // the parent has the style we want to apply and we are not trying to remove this style instead read: // the parent has the style we want to apply and we *are* trying to remove this style ?? Looks good otherwise.
Comment on attachment 64486 [details] [diff] [review] patch v1.0 r=jfrancis
Attachment #64486 - Flags: review+
Comment on attachment 64486 [details] [diff] [review] patch v1.0 email@example.com
Attachment #64486 - Flags: superreview+
Joe: no, no, my comment is ok. See the ! in the test.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
back out authorization requested from drivers
a=asa (on behalf of drivers) for checkin of backout.
Backed out (thanks a lot Asa, this was really needed) and reopened. See reasons why in bug 120377.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Removing mozilla0.9.8+ so things are less confusing for drivers trying to manage bug lists, since the bug is reopened but the reason it was + is fixed.
Status: REOPENED → ASSIGNED
Target Milestone: mozilla0.9.8 → mozilla0.9.9
minusing because we should focus on valid content, or content generated by us (which outta be valid)
Whiteboard: EDITORBASE 1 day → EDITORBASE- 1 day
Moving to Moz1.1
Target Milestone: mozilla0.9.9 → mozilla1.1
removing myself from the cc list
EDITORBASE+, msword compatibility
Whiteboard: EDITORBASE- 1 day → EDITORBASE+ 1 day [msword]
EDITORBASE+ topembed+ normalization
Daniel, Are you still working on this ?
On testcase in Comment 2 i get both Bold styles removed when pressing B button, so looks like WFM User agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0 SeaMonkey/2.13a1 Build identifier: 20120712003002
Assignee: daniel → nobody
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago → 6 years ago
QA Contact: bugzilla
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.