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

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
12 years ago
6 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
Posted file preview extension (obsolete) —
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
Posted patch patch (obsolete) — Splinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
(Assignee)

Comment 2

12 years ago
Posted patch patch v2 (obsolete) — Splinter Review
... 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

12 years ago
Posted patch patch v3 (obsolete) — Splinter Review
more optimized code and behaviour
Attachment #274022 - Attachment is obsolete: true
(Assignee)

Comment 4

12 years ago
Posted file preview extension (obsolete) —
Attachment #274017 - Attachment is obsolete: true
Attachment #274628 - Flags: ui-review?(beltzner)
Attachment #274017 - Flags: ui-review?(beltzner)
(Assignee)

Comment 5

12 years ago
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

12 years ago
Blocks: 154772, 386741
(Assignee)

Comment 6

12 years ago
Posted file preview extension
(Assignee)

Updated

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

Comment 7

12 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

12 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

12 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

12 years ago
No longer blocks: 386741
(Assignee)

Updated

12 years ago
Keywords: perf
(Assignee)

Updated

12 years ago
No longer blocks: 388030
(Assignee)

Comment 11

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

Updated

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