Closed Bug 99620 Opened 23 years ago Closed 22 years ago

[RFE] Color values for the Advanced Property editor should be in alphabetical order

Categories

(SeaMonkey :: Composer, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.9

People

(Reporter: TucsonTester2, Assigned: cmanske)

Details

Attachments

(1 file)

After Double-clicking on the body tag in "Show All Tags" view I highlighted the
bgcolor tag and clicked on the drop down menu in order to change the color value
to one of the options made available.  In here it was not so easy to find the
color I was looking for because they are not in alphabetical order.

Reproducibility: Every time

Steps to Reproduce:
1. Open Composer
2. Switch to the "Show all tags" view
3. Double click on the body tag
4. Highlight the bgcolor tag
5. Click on the value drop down box

Actual Results:
The colors will be out of alphabetical order

Expected Resutls:
I would expect that the colors would be in alpahebtical order in order to make
them easier to find.
Forgot build:
2001091003
-->cmanske
Assignee: brade → cmanske
Target Milestone: --- → mozilla1.0
Confirmed on Win 2k using build 20011001. I am changing this to a request for
enhancement, and updating the summary.
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: Color values for the Advanced Property editor should be in alphabetical order → [RFE] Color values for the Advanced Property editor should be in alphabetical order
load balancing
Status: NEW → ASSIGNED
Target Milestone: mozilla1.0 → mozilla0.9.9
Attached patch FixSplinter Review
Diff is weird. They are in alphabetical order now:
var gHTMLColors =
[
	"Aqua"
	"Black",
	"Blue",
	"Fuchsia",
	"Gray",
	"Green",
	"Lime",
	"Maroon",
	"Navy",
	"Olive",
	"Purple",
	"Red",
	"Silver",
	"Teal",
	"White",
	"Yellow",
];
Keywords: nsbeta1+, patch, review
Whiteboard: FIX IN HAND, need r=,sr=
Comment on attachment 68422 [details] [diff] [review]
Fix

sr=dveditz
Attachment #68422 - Flags: superreview+
checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: FIX IN HAND, need r=,sr=
Tucson/Michael, please verify this one..thanks.
I cannot verify this until bug 125115 is fixed.
Verified on trunk build 02-13.  

Note: there are still some problems getting the dropdown to come up, but that is
covered in bug 125115.
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: