47701 - UI: Advanced Edit dialog cleanup

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P4)

Description

[Simon Fraser [no longer active]]

Tree headers need background color or divider, to separate them from table 
contents

No way to delete an item from the tree; delete key does not work. Add Delete 
button.

Label fix: "Add _an_ HTML..."

Comment 1

[rubydoo123]

setting to nsbeta3+, result of the UI review

Comment 2

[leger]

beppe, it seem that not everything in this bug needs to be done.  Can you split 
into individual tasks?  Or a least outline minimal felt needed for PR3/RTM?  
Thanks!

Comment 3

[Charles Manske]

NO. It will all be done.
Leave my bugs alone, damn it.

Comment 4

[leger]

Updating [PDT 9/1] to [PDT Check]

Comment 5

[Charles Manske]

All requested work is checked in, but there is one small problem I need to
resolve with XPFE group.
Behavior notes:
When you click on an existing attribute in the "Name" column, the name should
be highlighted in a light gray or color, and the current "value" should be
selected. If you click in the value column, focus should be placed in the 
editbox, but an existing value should not be selected. The name to the left
of the value should show the selection highlighting -- that is the behavior
that is not working right now.
In previous version, you were able to select more than one item; now you can
select just one at a time. You were also able to right click (or Cmd+click
on Mac) for a popup menu which had "Remove Attribute" and "Add Attribute" 
items, but that was judged to be not very discoverable. To add a new item,
simply use the editboxes below the list. To remove, use the new "Remove
Attribute" button:
When a row is selected, clicking on the "Remove Attribute" button should delete
that item, and selection should shift to another item in the same row, or up
one row if you removed the last item in the list. Thus you can just click 
on the button repeatedly to easily remove multiple items.

Comment 6

[Charles Manske]

checked in 9/15
Problem with keeping focus when clicking on name still exist -- will file new
bug on that.

Comment 7

[sujay]

verified in 9/19 build.