Closed
Bug 389708
Opened 18 years ago
Closed 18 years ago
Don't switch to the plain Location Bar mode if the mouse is over the path
Categories
(Firefox :: Address Bar, defect)
Firefox
Address Bar
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: dao, Assigned: dao)
References
Details
(Keywords: ue)
Attachments
(2 files, 5 obsolete files)
15.91 KB,
patch
|
Details | Diff | Splinter Review | |
6.38 KB,
application/x-xpinstall
|
beltzner
:
ui-review-
|
Details |
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•18 years ago
|
||
Assignee: nobody → dao
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•18 years ago
|
||
... 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•18 years ago
|
||
more optimized code and behaviour
Attachment #274022 -
Attachment is obsolete: true
Assignee | ||
Comment 4•18 years ago
|
||
Attachment #274017 -
Attachment is obsolete: true
Attachment #274628 -
Flags: ui-review?(beltzner)
Attachment #274017 -
Flags: ui-review?(beltzner)
Assignee | ||
Comment 5•18 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•18 years ago
|
Assignee | ||
Comment 6•18 years ago
|
||
Assignee | ||
Updated•18 years ago
|
Attachment #274741 -
Flags: ui-review?(beltzner)
Assignee | ||
Comment 7•18 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 8•18 years ago
|
||
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•18 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•18 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 | ||
Comment 11•18 years ago
|
||
I've submitted most of the infrastructural fixes to bug 388030.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•