351620 - Margin Styles lost using advanced editor for inline styles

Status: RESOLVED DUPLICATE of bug 247816

(Core :: DOM: Editor, defect)

Description

[Joe Sabash [:JoeS1]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.5) Gecko/20060706 SeaMonkey/1.0.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.5) Gecko/20060706 SeaMonkey/1.0.3

Editing an element with an existing inline margin style will delete the margin property.

Reproducible: Always

Steps to Reproduce:
1.Compose an html document with composer or mailnews html compose
that has the following inline styles
<div
 style="border-style: solid; border-color: red; overflow: auto; width: 50%; margin-left: 10%;">text
in div</div>
This can be done by copy/paste into the insert html GUI in Mail or Composer.
2.Double click on the the div created above to bring up the Advanced Editor
3.Attempt to add an inline style, or remove a style (other than the margin)


Actual Results:  
The margin style will be removed by the edit action.

Expected Results:  
Margin style will be retained.

This happens with current Thunderbird (mailnews Compose), Seamonkey (mailnews or composer), and a 1 yr old copy of NVU.

Comment 1

[Joe Sabash [:JoeS1]]

I think this should be Core Editor, but don't see a way to assign it as such.
http://lxr.mozilla.org/seamonkey/source/editor/ui/dialogs/content/EdAdvancedEdit.xul#61
Seems to be a logical starting point.

Comment 2

[Joe Sabash [:JoeS1]]

The same symptoms seem to apply to to padding styles
I see some common code in 
https://bugzilla.mozilla.org/attachment.cgi?id=54673&action=view
Could this be related.

Comment 3

[Joe Sabash [:JoeS1]]

Here is a screenshot of the the Advanced editor window that could provide a clue.
http://img241.imageshack.us/img241/3084/advancededituz8.gif
I believe the property name should be margin-left and not margin-left-value
The same thing happens on margin-left, margin-right, padding-left and padding-right
When the the tree is rebuilt after the edit, the munged property name is rejected.

Comment 4

[Mike Cowperthwaite]

Reproduced in NVu 1.0, TB 3a1-0907, Win2K.

Comment 5

[Boris Zbarsky [:bzbarsky]]

I believe we have existing bugs on this...

Comment 6

[Joe Sabash [:JoeS1]]

Duplicate but the scope should be expanded.

*** This bug has been marked as a duplicate of 247816 ***