xul has confusing references to "Advanced" ("More Attribs")

VERIFIED FIXED in M9

Status

()

Core
Editor
P3
normal
VERIFIED FIXED
19 years ago
16 years ago

People

(Reporter: Kathleen Brade, Assigned: Charles Manske)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
The Link dialog has referenced to a titledbutton with id="Advanced" which is
intended to be the "More Attributes..." (previously known as "Extra HTML").  The
id and onclick handler should be renamed.

The HRule and NamedAnchor dialogs also have such titledbuttons which should have
their ids and onclick handlers renamed.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M9
(Assignee)

Comment 1

19 years ago
I'm sorry about the IDs -- I didn't realize they were the source of the
confusion between "Advanced" and "More Attributes"
We need both buttons for those dialogs that will have an "Advanced" expanded
section: Image, Table/Cell, and Link dialogs, although I'm not sure if the
link dialog really needs it.
We need to resolve where to put each in the dialogs.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

19 years ago
We have decided to use "Advanced Edit..." instead of "More Attributes..." for
the button, and although I changed the IDs and onclick handler names, they
should be changed to be consistent with the action.
We will use "More" and "Fewer" as the label for the button that
expands/contracts a dialog to show/hide controls.
Kathy: what would you suggest for the name of that button and onclick handler?
How about "MoreOrLess" for the ID and "onMoreOrLess" for the onclick?
(Reporter)

Comment 3

19 years ago
that's fine with me

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 4

19 years ago
verified in 7/28 build.
(Reporter)

Comment 5

19 years ago
The link dialog still has an entity declared as "Advanced" when it should be
"More" (and another one declared as "Fewer")
You need to log in before you can comment on or make changes to this bug.