Clicking on miscellaneous URLbar icons selects the URLbar text

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
UI Design
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: neil@parkwaycc.co.uk)

Tracking

Trunk
seamonkey2.1b2

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
Steps to reproduce the problem:
1. Configure preferences for selection of URLbar text
(browser.urlbar.clickSelectsAll set to true)
2. Move focus away from the URL bar
3. Click on any of the miscellaneous URLbar icons:
   Feed icon
   EV icon
   various doorhanger icons

Actual result: URLbar text is selected and focused

Expected result: URLbar text is unaffected

Additional information: The Places Start Button has a partial workaround.
(Assignee)

Comment 1

6 years ago
Created attachment 497760 [details] [diff] [review]
Possible patch

Option 1: Ignore clicks that aren't actually on the URL itself.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #497760 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 2

6 years ago
Created attachment 497761 [details] [diff] [review]
Possible patch
[Checked in: Comment 4]

Option 2: Prevent mouse events on miscellaneous icons from propagating.
Attachment #497761 - Flags: review?(iann_bugzilla)
(Assignee)

Updated

6 years ago
Blocks: 617333

Updated

6 years ago
Attachment #497760 - Flags: review?(iann_bugzilla) → review+

Comment 3

6 years ago
Comment on attachment 497761 [details] [diff] [review]
Possible patch
[Checked in: Comment 4]

r=me I think my preference is for this one but there is not much in that, so whichever you prefer.
Attachment #497761 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 4

6 years ago
Comment on attachment 497761 [details] [diff] [review]
Possible patch
[Checked in: Comment 4]

Pushed changeset 57e1d4eca307 to comm-central.
(Assignee)

Updated

6 years ago
Attachment #497760 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Attachment #497761 - Attachment description: Possible patch → Possible patch [Checked in: Comment 4]
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.