Closed Bug 120120 Opened 23 years ago Closed 23 years ago

horizontal line properties window does not show that line is centered

Categories

(SeaMonkey :: Composer, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: TucsonTester1, Assigned: neil)

Details

Attachments

(1 file)

build: 2002011103
platform: both Windows 2000 and Windows ME
reproduce: always

1. Open composer
2. create a horizontal line by clicking on the horizontal line button on the
tool bar.
3. Double-click on the new horizontal line and change dimentions to "pixels"
instead of "% of window"
4. In alignment, choose center, and click OK
5. After line is centered and modified in size, double-click on horizontal line
again to get the horizontal properties window.  Notice that the alignment is
showing as left and not center, even though the line is still centered in composer.

Expected results: In the horizontal line properties menu, the "center" alignment
button should be selected.

Actual results: Whenever a horizontal line is centered, it displays it as left
aligned in the horizontal properties window, even if it is still centerd. 
Changing size isn't necessary.  Both left and right aligned display properly,
just not the "center" option.
Attached patch Proposed patchSplinter Review
CC'ing cmanske and glazou for review.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: patch, review
Comment on attachment 65428 [details] [diff] [review]
Proposed patch

Good catch, Neil. This should go into 098.
Ask sr= from kin.
Attachment #65428 - Flags: review+
forgot to say r=glazman but it was added to patch manager
Neil, this bug is yours.
Assignee: syd → neil
Comment on attachment 65428 [details] [diff] [review]
Proposed patch

sr=kin@netscape.com
Attachment #65428 - Flags: superreview+
a=blizzard on behalf of drivers for 0.9.8
Keywords: mozilla0.9.8+
checked in
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified.  Looks good to me on Win 2k using trunk build 2002012103.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: