Properties context menu option in Bookmarks Manager does nothing when the embedded properties pane is already open

RESOLVED FIXED in seamonkey2.7

Status

--
trivial
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: neil, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.7

SeaMonkey Tracking Flags

(seamonkey2.5 wontfix, seamonkey2.6 fixed, seamonkey2.7 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
I came up with a number of potentially acceptable behaviours:
* Toggle the More/Fewer button if it is visible
* Hide the menuitem when the properties are displayed
* Turn the menuitem into a checkbox toggle that hides the properties again
  (thus making the splitter accessible)
Or maybe you could think of something better.
(Assignee)

Comment 1

7 years ago
* Toggling the More button is unexpected/inconsistent with what Properties does when the pane is collapsed, so this is not a good option.
* Hiding/disabling the menuitem is a valid option.
* Turning it into a checkbox doesn't sound like an improvement to me.

I'd suggest to just focus the first text field (Name); basically the same as pressing Tab. That will select the contents of the text field, which is quite visible (unless the field is empty). We could even do that unconditionally.
(Assignee)

Comment 2

7 years ago
Created attachment 569716 [details] [diff] [review]
focus Name [Checkin: comments 3 and 5]

Every item in the BM (except for separators, which do not have Properties enabled) has at least Name, which is always first. The attached patch focuses the corresponding text box. On Windows, the text inside the box will be selected, too; on Linux the activation is not so obvious (only focus ring and caret appear) but still enough IMHO.

Not taking until I get at least f+.
Attachment #569716 - Flags: review?(neil)
(Reporter)

Updated

7 years ago
Attachment #569716 - Flags: review?(neil) → review+
(Assignee)

Updated

7 years ago
Assignee: nobody → jh
Status: NEW → ASSIGNED
(Assignee)

Comment 3

7 years ago
Comment on attachment 569716 [details] [diff] [review]
focus Name [Checkin: comments 3 and 5]

http://hg.mozilla.org/comm-central/rev/806affdb379c
Attachment #569716 - Attachment description: focus Name → focus Name [Checkin: comment 3]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.7
(Assignee)

Comment 4

7 years ago
Comment on attachment 569716 [details] [diff] [review]
focus Name [Checkin: comments 3 and 5]

Base bug 674208 already landed on Aurora and Beta, and this is a small and safe improvement to that one. Probably too late for Beta, so just asking for Aurora approval.
Attachment #569716 - Flags: approval-comm-aurora?

Updated

7 years ago
Attachment #569716 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 5

7 years ago
Comment on attachment 569716 [details] [diff] [review]
focus Name [Checkin: comments 3 and 5]

http://hg.mozilla.org/releases/comm-aurora/rev/8067ee8641fc
Attachment #569716 - Attachment description: focus Name [Checkin: comment 3] → focus Name [Checkin: comments 3 and 5]
(Assignee)

Updated

7 years ago
status-seamonkey2.5: --- → wontfix
status-seamonkey2.6: --- → fixed
status-seamonkey2.7: --- → fixed
You need to log in before you can comment on or make changes to this bug.