Don't switch to the plain Location Bar mode if the mouse is over the path

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
12 years ago
5 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

({ue})

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 5 obsolete attachments)

(Assignee)

Description

12 years ago
Created attachment 274017 [details]
preview extension

This will make the host "move" less often and should thus reduce the overall noise of the new Location Bar.
Attachment #274017 - Flags: ui-review?(beltzner)
(Assignee)

Comment 1

12 years ago
Created attachment 274018 [details] [diff] [review]
patch
Assignee: nobody → dao
Status: NEW → ASSIGNED
(Assignee)

Comment 2

12 years ago
Created attachment 274022 [details] [diff] [review]
patch v2

... and switch on hover only if the protocol is hidden. This doesn't really matter for ui-review.
Attachment #274018 - Attachment is obsolete: true
(Assignee)

Comment 3

11 years ago
Created attachment 274627 [details] [diff] [review]
patch v3

more optimized code and behaviour
Attachment #274022 - Attachment is obsolete: true
(Assignee)

Comment 4

11 years ago
Created attachment 274628 [details]
preview extension
Attachment #274017 - Attachment is obsolete: true
Attachment #274628 - Flags: ui-review?(beltzner)
Attachment #274017 - Flags: ui-review?(beltzner)
(Assignee)

Comment 5

11 years ago
Created attachment 274740 [details] [diff] [review]
patch v4, kill <scrollbox>

This means that the tooltip with the plain URL will be displayed even if the URL doesn't overflow the Location Bar, and URLs that are too long for the tooltip will be cropped at the center rather than at the start, in order to expose the scheme.
Attachment #274627 - Attachment is obsolete: true
Attachment #274628 - Attachment is obsolete: true
Attachment #274628 - Flags: ui-review?(beltzner)
(Assignee)

Updated

11 years ago
Blocks: 154772, 386741
(Assignee)

Comment 6

11 years ago
Created attachment 274741 [details]
preview extension
(Assignee)

Updated

11 years ago
Attachment #274741 - Flags: ui-review?(beltzner)
(Assignee)

Comment 7

11 years ago
Note that getting rid of <scrollbox> could help performance a bit, which is why I've added this bug to the dependency list of bug 386741.
Keywords: perf
Comment on attachment 274741 [details]
preview extension

I'm not sure that this is actually better, since it still jumps when you put focus in the field, but now only sometimes jumps when you hover over the field.

It's an interesting idea, but the inconsistency in the behaviour makes it even more confusing, IMO.
Attachment #274741 - Flags: ui-review?(beltzner) → ui-review-
(Assignee)

Comment 9

11 years ago
(In reply to comment #8)
> (From update of attachment 274741 [details])
> I'm not sure that this is actually better, since it still jumps when you put
> focus in the field, but now only sometimes jumps when you hover over the field.

You actually have to halt the mouse over the host. With this change, simply moving the mouse over the field wouldn't trigger the switch in any case.

Other than that, I can't add much. I have a different opinion ...
(Assignee)

Comment 10

11 years ago
(In reply to comment #8)
> (From update of attachment 274741 [details])
> I'm not sure that this is actually better, since it still jumps when you put
> focus in the field, but now only sometimes jumps when you hover over the field.
> 
> It's an interesting idea, but the inconsistency in the behaviour makes it even
> more confusing, IMO.

So, any idea how to improve this?
(Assignee)

Updated

11 years ago
No longer blocks: 386741
(Assignee)

Updated

11 years ago
Keywords: perf
(Assignee)

Updated

11 years ago
No longer blocks: 388030
(Assignee)

Comment 11

11 years ago
I've submitted most of the infrastructural fixes to bug 388030.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → WONTFIX

Updated

5 years ago
No longer blocks: 154772
You need to log in before you can comment on or make changes to this bug.