Closed Bug 389708 Opened 14 years ago Closed 13 years ago

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

Categories

(Firefox :: Address Bar, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: dao, Assigned: dao)

References

Details

(Keywords: ue)

Attachments

(2 files, 5 obsolete files)

Attached 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)
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attached 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
Attached patch patch v3 (obsolete) — Splinter Review
more optimized code and behaviour
Attachment #274022 - Attachment is obsolete: true
Attached file preview extension (obsolete) —
Attachment #274017 - Attachment is obsolete: true
Attachment #274628 - Flags: ui-review?(beltzner)
Attachment #274017 - Flags: ui-review?(beltzner)
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)
Blocks: 154772, 386741
Attached file preview extension
Attachment #274741 - Flags: ui-review?(beltzner)
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-
(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 ...
(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?
No longer blocks: 386741
Keywords: perf
No longer blocks: 388030
I've submitted most of the infrastructural fixes to bug 388030.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
No longer blocks: 154772
You need to log in before you can comment on or make changes to this bug.