Closed
Bug 594050
Opened 14 years ago
Closed 7 years ago
Hide go button when url bar is blank in blank tab
Categories
(Firefox :: Theme, defect)
Tracking
()
People
(Reporter: alex_mayorga, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
fryn
:
feedback-
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows NT 6.0; rv:2.0b6pre) Gecko/20100907 Firefox/4.0b6pre
Build Identifier: Mozilla/5.0 (Windows NT 6.0; rv:2.0b6pre) Gecko/20100907 Firefox/4.0b6pre
The button is green even when there's nothing typed in URL bar.
Confusing because clicking GSR button when the URL bar is empty won't go anywhere.
Reproducible: Always
Steps to Reproduce:
1. Open a new tab or window(show a blank page).
Actual Results:
The button is green even when there's nothing typed in URL bar.
Expected Results:
IMHO button should turn green once there's, at least, one typed character in the URL bar.
Comment 1•14 years ago
|
||
I just followed the Firefox 3.6 behavior when implementing this. The go button appeared whenever the url bar textbox had been edited or did not match the current url, regardless of its emptiness.
UX team, input?
OS: Windows Vista → All
Hardware: x86 → All
Updated•14 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•14 years ago
|
||
I think original behavior makes sense. What else are we gonna put in place of it? Reload? Stop? Both of those make even less sense. At least the Go button makes it seem like you're about to "Go" somewhere with the new tab and Fx is waiting for you.
Reporter | ||
Comment 3•14 years ago
|
||
How about a "grayed out go" when the URL bar is empty or unchanged?
Another behavior I find funny is coming back to a new tab/window with an empty URL bar, you'd be greeted by a "reload" button that wouldn't do anything either.
I think this also can be prevented if the empty/unchanged case is properly handled.
Comment 4•14 years ago
|
||
(In reply to comment #3)
> How about a "grayed out go" when the URL bar is empty or unchanged?
> Another behavior I find funny is coming back to a new tab/window with an empty
> URL bar, you'd be greeted by a "reload" button that wouldn't do anything
> either.
> I think this also can be prevented if the empty/unchanged case is properly
> handled.
This will be fixed in bug 565575 by making sure the url bar is focused. Displaying a disabled (ghosted) go button makes sense to me.
Comment 5•14 years ago
|
||
I also think the way it works makes sense and is fine.
Comment 7•14 years ago
|
||
>The go button
>appeared whenever the url bar textbox had been edited or did not match the
>current url, regardless of its emptiness.
>
>UX team, input?
Having the button appear on first edit should help users understand what it is for more. After they type a single character the button will appear, and that sequence means "I can perform an action on the thing you just entered." Also, I agree with comment #0 that hitting go when everything is blank is a bit strange.
Comment 8•14 years ago
|
||
(In reply to comment #7)
> Having the button appear on first edit should help users understand what it is
> for more. After they type a single character the button will appear, and that
> sequence means "I can perform an action on the thing you just entered." Also,
> I agree with comment #0 that hitting go when everything is blank is a bit
> strange.
Agreed. It shouldn't indicate that something works when it doesn't do anything. Go button being ghosted/white bg until something has been entered makes sense, combined with the fix in bug 565575 (adding as dependency for this one).
Depends on: 565575
Comment 9•14 years ago
|
||
Also note that while a new tab shows the green go button right after it is created, switching to another tab and back to the new one (with nothing inside the entry) now shows the reload button
Comment 10•14 years ago
|
||
Comment 11•14 years ago
|
||
>Go button being ghosted/white bg until something has been entered makes sense
We are using this appearance for active but not hovered (like reload), so I think the best way to go would be for the go button to simply not exist when it is inactive.
Comment 12•14 years ago
|
||
(In reply to comment #11)
> We are using this appearance for active but not hovered (like reload), so I
> think the best way to go would be for the go button to simply not exist when it
> is inactive.
What would be displayed then, if neither go, reload, nor stop doesn't make sense?
Comment 13•14 years ago
|
||
we would display a very zen nothingness (well, the drop down arrow would still be there until the user entered something)
Reporter | ||
Comment 14•14 years ago
|
||
(In reply to comment #13)
> we would display a very zen nothingness (well, the drop down arrow would still
> be there until the user entered something)
+1
Paired with bug 587901, comment 50 and we'd end up with a beautiful empty "awesome bar" =)
Comment 15•14 years ago
|
||
(In reply to comment #11)
> >Go button being ghosted/white bg until something has been entered makes sense
>
> We are using this appearance for active but not hovered (like reload), so I
> think the best way to go would be for the go button to simply not exist when it
> is inactive.
I guess there are now 5 different states of go/stop/reload button:
1. Normal
2. Hover
3. Pressed
and these three are from this mockup: https://wiki.mozilla.org/images/8/82/Firefox-4-Mockup-i05-(Win7)-(Aero)-(TabsTop)-(ButtonStates).png
4. Reload with white bg when not hovered
5. Grayed stop for a short time after loading a page
I suggest to make grayed go button, analogous to the grayed stop.
Comment 16•14 years ago
|
||
(In reply to comment #13)
> we would display a very zen nothingness (well, the drop down arrow would still
> be there until the user entered something)
I'll make a patch for this.
Assignee: nobody → fryn
Status: NEW → ASSIGNED
Version: unspecified → Trunk
Comment 17•14 years ago
|
||
Unassigning myself, since I've got a ton on my plate already.
Looks like a good second bug ;P
Assignee: fryn → nobody
Status: ASSIGNED → NEW
Comment 18•14 years ago
|
||
(In reply to comment #11)
> >Go button being ghosted/white bg until something has been entered makes sense
>
> We are using this appearance for active but not hovered (like reload), so I
> think the best way to go would be for the go button to simply not exist when it
> is inactive.
That's what I meant by saying "white bg" — sorry for the lack of clarity here. So we agree completely, the button shouldn't show up at all until some text has been entered.
Updated•14 years ago
|
Assignee: nobody → margaret.leibovic
Comment 19•14 years ago
|
||
Attachment #498901 -
Flags: feedback?(fryn)
Comment 20•14 years ago
|
||
Comment on attachment 498901 [details] [diff] [review]
patch
Broken cases:
1. Have some text in the url bar.
2. Press ctrl/cmd+t.
3. Go button is there but shouldn't be.
1. Have some text in the url bar.
2. Select all; cut.
3. Go button is there but shouldn't be.
It seems like onInput does not cover all the cases we need.
Getting it right in only some cases makes it unpredictable, which is worse than the current experience.
FWIW, the disappearance of the button feels quite jumpy. :\
Let's get UX people to try this out first-hand.
Attachment #498901 -
Flags: feedback?(fryn) → feedback-
Comment 21•14 years ago
|
||
Correction:
1. Have some text in the url bar.
2. Click in the url bar to focus the textbox.
3. Press ctrl/cmd+t.
4. Go button is there but shouldn't be.
Reporter | ||
Comment 22•14 years ago
|
||
(In reply to comment #20)
> Comment on attachment 498901 [details] [diff] [review]
> patch
Anyway I can help to review this?
Comment 23•14 years ago
|
||
(In reply to comment #22)
> (In reply to comment #20)
> > Comment on attachment 498901 [details] [diff] [review]
> > patch
>
> Anyway I can help to review this?
The patch doesn't work properly, as Frank pointed out above, so it still needs work before it can be reviewed.
I stopped working on this because it's not blocking Firefox 4. If you have time and energy to work on it, feel free to take it from me! :)
Reporter | ||
Comment 24•14 years ago
|
||
(In reply to comment #23)
> I stopped working on this because it's not blocking Firefox 4. If you have
> time and energy to work on it, feel free to take it from me! :)
I probably have the energy, but not the time nor the skill =(
In any case, if you're not actively pursuing this, can you please release the "Assigned To:"?
Comment 25•14 years ago
|
||
(In reply to comment #24)
> (In reply to comment #23)
> > I stopped working on this because it's not blocking Firefox 4. If you have
> > time and energy to work on it, feel free to take it from me! :)
>
> I probably have the energy, but not the time nor the skill =(
> In any case, if you're not actively pursuing this, can you please release
> the "Assigned To:"?
I'm unassigning myself, but you could have just done that by making yourself the assignee :) If you decide to work on this and find you have trouble, you can ask me questions.
Assignee: margaret.leibovic → nobody
Reporter | ||
Comment 26•14 years ago
|
||
(In reply to comment #25)
> I'm unassigning myself, but you could have just done that by making yourself
> the assignee :) If you decide to work on this and find you have trouble, you
> can ask me questions.
If my "bugzilla fu" is that low, imagine my "hacking fu" =)
I've filed bug 656002 that might be related as "Reload Tab" is visible on about:blank and about:home pages that as far as I understand are static and shouldn't show a reload option IMHO.
Reporter | ||
Comment 27•14 years ago
|
||
How can I put this one on http://areweprettyyet.com/4/mainWindow/ ?
Comment 28•14 years ago
|
||
this is more of an interactive bug than visual polish issue, but either way we should get around to fixing it.
Updated•13 years ago
|
No longer blocks: 593293
Summary: Go/Stop/Reload (GSR) button green on new tab(s)/window(s) → Hide go button when url bar is blank in blank tab
Comment 30•13 years ago
|
||
Now that there's no more big green button, this doesn't strike me as much of a problem anymore (it's definitely not distracting). When we were working on this before, I remember making the button disappear felt jumpy, so I propose we WONTFIX this.
Reporter | ||
Comment 31•13 years ago
|
||
The fact is the "button" is still there and does nothing.
Also if you move the focus from the URL bar to anywhere in a "new tab" the go button changes to a "reload" one that still does nothing.
Same for the new "globe" on the left of the URL bar.
All this "odd" behavior might be misconstrued as "broken" as you're giving users affordances that do nothing.
IMHO the only affordance that makes sense in the URL bar on about:newtab is the down arrow as it actually does something.
I believe it "felt jumpy" because the icons in the URL bar are ordered "incorrectly" IMHO.
Icons on the right hand side of the URL bar should be ordered go/reload, favorite star, down arrow.
Go appears only when there's something typed in the URL bar and is the one closer to the typed URL as it allows one to act on it.
Favorite star only appears when there's a page loaded, this already appears "magically" today.
Down arrow is in there permanently on the right hand side as it is usable on every state of the URL bar and also resembles a "combo box" that gives the user access to the "world wide web".
Hope it all made sense =)
Reporter | ||
Comment 32•12 years ago
|
||
Updated the summary to reflect comment 31 and added some blocks that I believe make sense.
Blocks: 580046, cuts-control
Flags: needinfo?
Summary: Hide go button when url bar is blank in blank tab → "Globe" and "Go" buttons in "New Tab" do nothing and should not appear in UI until the affordances are useable
Comment 33•12 years ago
|
||
Comment 31 are just random thoughts about loosely related stuff.
No longer blocks: cuts-control, 580046
Flags: needinfo?
Summary: "Globe" and "Go" buttons in "New Tab" do nothing and should not appear in UI until the affordances are useable → Hide go button when url bar is blank in blank tab
Updated•11 years ago
|
Blocks: fxdesktopbacklog
Updated•11 years ago
|
Whiteboard: p=0
Updated•11 years ago
|
No longer blocks: fxdesktopbacklog
Flags: firefox-backlog+
Updated•11 years ago
|
Whiteboard: p=0 → p=2
Updated•10 years ago
|
Points: --- → 2
Flags: qe-verify?
Whiteboard: p=2
Reporter | ||
Updated•9 years ago
|
status-firefox40:
--- → affected
status-firefox41:
--- → affected
status-firefox42:
--- → affected
status-firefox43:
--- → affected
Version: Trunk → 43 Branch
Reporter | ||
Comment 34•7 years ago
|
||
¡Hola!
I believe https://bugzilla.mozilla.org/show_bug.cgi?id=1346488 fixed this somewhat accidentally.
¡Gracias!
Alex
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → affected
status-firefox57:
--- → fixed
status-firefox58:
--- → fixed
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•