Closed Bug 102036 Opened 24 years ago Closed 24 years ago

Image properties dialog should use "middle", not "center" for align attribute.

Categories

(SeaMonkey :: Composer, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.6

People

(Reporter: TucsonTester2, Assigned: cmanske)

Details

Attachments

(1 file)

Build ID: 20010924 If you go into the advanced property editor for images it does not display the correct value for the option chosen. Steps To Reproduce: 1.Open Composer 2.Click on the image button on the toolbar 3.Choose an image and click on the advanced edit button 4.Click on alignment and then click on the value drop-down menu Actual Results: The values middle and bottom will be in the window. The actual values that should be there are center and baseline. The center does appear if you choose the alignment in the image properties and then go into advanced edit. Expected Results: Since it is the "Advanced" property editor I would expect to see the html tags actual values in there. This way it should be shown is demonstrated in list properties. The solid circle description is in the properties menu but the disc value is in the Advanced Property editor.
-->cmanske
Assignee: syd → cmanske
Target Milestone: --- → mozilla0.9.6
From the W3C definition of "align" (http://www.w3.org/TR/REC-html40/struct/objects.html#adef-align-IMG): align = bottom|middle|top|left|right Deprecated. This attribute specifies the position of an IMG, OBJECT, or APPLET with respect to its context. So the image dialog is wrong, not the Advanced Properties dialog. Chaning summary to reflect that.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Summary: Advanced Property Editor alignment values for images needs to be changed → Image properties dialog should use "middle", not "center" for align attribute.
Attached patch Fix.Splinter Review
Keywords: patch, review
Whiteboard: FIX IN HAND need r=,sr=
Comment on attachment 54495 [details] [diff] [review] Fix. r=brade
Attachment #54495 - Flags: review+
Whiteboard: FIX IN HAND need r=,sr= → FIX IN HAND need sr=
Comment on attachment 54495 [details] [diff] [review] Fix. sr=hewitt
Attachment #54495 - Flags: superreview+
checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: FIX IN HAND need sr=
Verified on build 2001110903.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: