My favorites | Sign in
Project Home Downloads Wiki Issues
New issue   Search
for
  Advanced search   Search tips
Issue 45745: Remove omnibox ears.
23 people starred this issue and may be notified of changes. Back to list
 
Reported by project member ben@chromium.org, Jun 3, 2010
We should detach the ears from the omnibox like so:

- Reload becomes its own button again and combines with Stop. I will not stab 
anyone if this is done the way it's combined with Go now. (i.e. using logic 
so it doesn't change state if the mouse is over it).
- We kill the go button.
- The omnibox gets rounded rect edges matching the shape of adjacent buttons 
on either end.


Comment 1 by ben@chromium.org, Jun 3, 2010
(No comment was entered for this change.)
Labels: OS-Windows
Comment 2 by shess@chromium.org, Jun 3, 2010
Q: Does the popup go back to rounded-top with a gap?
Comment 3 by pkasting@chromium.org, Jun 3, 2010
A: Yes.  But we don't need to re-add the PopupPositioner since it's still only the 
width of the omnibox.

Of course, this does mean I need to hack up the info bubble code more.  Urgh.
Comment 4 by RobinBrightEyes@gmail.com, Jun 4, 2010
- I agree, but you could try just removing the right ear and combine the go and stop 
into the reload button on left ear. Rounding the edges of the omnibox on right side.

- I like the way the reload is attached to the omnibox, it looks neat and it shows that 
they are connected in the literal sense.

-The navigation forward and back buttons are on the left so makes sense to have the new 
combined go, stop reload button on left anyway.
Comment 5 by fabios4rulez, Jun 4, 2010
can we see some mocks for the desired implementation here? thanks...
Comment 6 by jecsh...@gmail.com, Jun 4, 2010
I want to see how this goes. Especially my usability, since I'm used to WinExplorer and 
IE with their stop/reload on the right ear. And now I prefer using the Go button for 
everything:
1, Visit the typed text
2. Stop loading
3. Reload the page
Comment 7 by est...@chromium.org, Jun 4, 2010
(No comment was entered for this change.)
Labels: Restrict-AddIssueComment-Commit
Comment 8 by pkasting@chromium.org, Jun 14, 2010
This is basically done in r49727.  I still need correct button images, but I claim that can be its own bug.
Status: Fixed
Sign in to add a comment

Powered by Google Project Hosting