Closed
Bug 615435
Opened 14 years ago
Closed 14 years ago
Relocate the Home button to the right side of the browser
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
VERIFIED
FIXED
Firefox 4.0b8
Tracking | Status | |
---|---|---|
blocking2.0 | --- | - |
People
(Reporter: faaborg, Assigned: fryn)
References
Details
(Whiteboard: [target-next-beta])
Attachments
(2 files, 3 obsolete files)
8.94 KB,
patch
|
dao
:
review+
Dolske
:
approval2.0+
|
Details | Diff | Splinter Review |
203.85 KB,
image/png
|
Details |
This change is likely to be controversial, but I believe we should relocate the Home button to be to the left of the bookmarks button. This button was never meant to be part of Firefox 4 (we were transitioning to the home tab), so the fact that it is still in our UI at this stage is a bit unplanned. Cons: -Breaks consistency with previous versions of Firefox -Will move to the left again in Firefox 4.x or 5 when we introduce the home tab, which is a bit chaotic Pros: -External consistency with the bulk of non-Firefox market share (IE7/8/9) -Nice mapping with the bookmarks menu (home is basically a super bookmark) -Users who hit home in order to do a google search (most of them) will potentially discover the search bar, which is now right next to the home button -No consistency violation with Chrome (doesn't have a home button by default)* -No consistency violation with Safari (doesn't have a home button) -Reduces the visual complexity of the back/forward area back to the original Firefox 4 mockups Mostly it feels like the arguments not to move the control mostly come down to trying to reducing how much we change our UI. Users can of course always customize the button position to wherever they would like to place it. [* depending on what day of the week it is, and how the stars are aligned. I really can't figure out their position on the home button.]
Reporter | ||
Comment 1•14 years ago
|
||
It would be good to get this change in for beta 8 if possible, so that it is just late, instead of super late, in the ship cycle.
blocking2.0: --- → ?
Reporter | ||
Updated•14 years ago
|
Summary: Relocate the Home button to the left side of the browser → Relocate the Home button to the right side of the browser
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → fryn
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•14 years ago
|
||
This moves the home button between the search box and the bookmarks menu button in the default configuration. If the navigation toolbar has been customized, it performs a one-time migration, moving the home button (if it exists) to the immediate left of the bookmarks menu button.
Attachment #494502 -
Flags: ui-review?(limi)
Attachment #494502 -
Flags: review?(dao)
Comment 3•14 years ago
|
||
Comment on attachment 494502 [details] [diff] [review] patch Looks good to me, I agree that we should ship a beta with this and see how it works out. External consistency with the only browser that ships with a Home button (IE) makes sense, and it also makes the UI cleaner.
Attachment #494502 -
Flags: ui-review?(limi) → ui-review+
Comment 4•14 years ago
|
||
Comment on attachment 494502 [details] [diff] [review] patch >+ currentset = currentset.replace(/(^|,)home-button($|,)/, "$1$2"). >+ replace(/(^|,)bookmarks-menu-button-container($|,)/, Nit: move the trailing dot to the start of the second line: currentset = currentset.replace(...) .replace(...
Attachment #494502 -
Flags: review?(dao) → review+
Updated•14 years ago
|
Component: General → Toolbars
QA Contact: general → toolbars
Assignee | ||
Comment 5•14 years ago
|
||
Gah, missed an edge case: When the bookmarks menu button has been customized away (manually dragged out of the navigation toolbar, not just by clicking View Bookmarks Toolbar), where should the home button go? 1. The home button is like a gigantic bookmark, so remove it too. (Current patch does this.) 2. Move it to the rightmost end of the navigation toolbar.
Comment 6•14 years ago
|
||
We shouldn't just remove it. Those who've removed the bookmarks menu button would have been capable of removing the home button.
Assignee | ||
Comment 7•14 years ago
|
||
Attachment #494960 -
Flags: review?(dao)
Assignee | ||
Updated•14 years ago
|
Attachment #494502 -
Attachment is obsolete: true
Assignee | ||
Comment 8•14 years ago
|
||
In that case, let's go with this slightly modified patch. It simply moves the home button to the rightmost end of the navigation toolbar, if the bookmarks menu button had been manually removed.
Comment 9•14 years ago
|
||
Comment on attachment 494960 [details] [diff] [review] patch v2 >+ currentset = currentset += ",home-button"; This moves the home button after fullscreenflex and window-controls.
Attachment #494960 -
Flags: review?(dao) → review-
Comment 10•14 years ago
|
||
(And I assume you meant currentset += ",home-button"...)
Assignee | ||
Comment 11•14 years ago
|
||
Now accounts for fullscreenflex.
Attachment #494960 -
Attachment is obsolete: true
Attachment #494961 -
Flags: review?(dao)
Comment 12•14 years ago
|
||
Comment on attachment 494961 [details] [diff] [review] patch v3 fullscreenflex isn't removable, so the last branch should never be entered. Can you just do nothing if bookmarks-menu-button-container isn't found?
Attachment #494961 -
Flags: review?(dao) → review-
Assignee | ||
Comment 13•14 years ago
|
||
(In reply to comment #12) > Comment on attachment 494961 [details] [diff] [review] > patch v3 > > fullscreenflex isn't removable, so the last branch should never be entered. > > Can you just do nothing if bookmarks-menu-button-container isn't found? If you look at the migration code for adding the bookmarks menu button, there's a branch for the case where fullscreenflex isn't there. I suppose if we migrate from a very old version, then we may hit that. I don't know when fullscreenflex was added. Considering that the main reasons for moving the home button are not really valid if the user already knew how to remove the bookmarks menu button, we can just do nothing in that case. Patch coming up.
Version: unspecified → Trunk
Assignee | ||
Comment 14•14 years ago
|
||
Attachment #494961 -
Attachment is obsolete: true
Attachment #494965 -
Flags: review?(dao)
Updated•14 years ago
|
Attachment #494965 -
Flags: review?(dao) → review+
Assignee | ||
Updated•14 years ago
|
Attachment #494965 -
Flags: approval2.0?
Comment 15•14 years ago
|
||
>Cons:
>-Breaks consistency with previous versions of Firefox
Counts four times in my opinion.
Why moving it to the right if you plan to move it to the left again, this time as a home-tab? People got used to its current positions.. I for instance moved the home button to the left of the tab-bar just to get used to its new place. Putting it on the right that far tho, is not helpful at all I think.
Reporter | ||
Comment 16•14 years ago
|
||
>this time as a home-tab?
I should note that the Home Tab will behave differently than the current home button, since it will be associated with Firefox specific UI. The current notion of "home page" will be the which tab is shown when the browser launches (so creating a home page is essentially the same thing as creating a new app tab, that will be shown on launch).
Comment 17•14 years ago
|
||
Now that you have decided to move the home button to the right ("home is basically a super bookmark"), you could merge it with the bookmarks button (they do similar function)
Updated•14 years ago
|
Attachment #494965 -
Flags: approval2.0? → approval2.0+
Updated•14 years ago
|
Flags: in-litmus?
Assignee | ||
Comment 18•14 years ago
|
||
Pushed. http://hg.mozilla.org/mozilla-central/rev/7284451662bf
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
blocking2.0: ? → ---
Target Milestone: --- → Firefox 4.0b8
Updated•14 years ago
|
Whiteboard: [target-next-beta]
Comment 19•14 years ago
|
||
Just FTR, if this regresses, we'll backout before we fix.
blocking2.0: --- → -
Comment 20•13 years ago
|
||
Bug 640531 - Home Button needs to be on the Left of URL Bar by Default Home Button needs to be on the Left hand side of URL Bar by Default. No need to frustrate non technical users by changing location to right hand side. Non technical users will not know that it is customizable, View ==> Toolbars ==> Customize, as it is not apparent unless you have read somewhere (instructions). That you can drag and drop other stuff i.e. home button, as it is not in the window which opens when you click customize. There are 4,056 Messages when you search "home" in the Beta feedback: http://input.mozilla.com/en-US/beta/search?product=firefox&q=home , and most are confused why it was moved to the right.
Comment 21•13 years ago
|
||
Why FF developers always moving around all toolbar buttons? Put all back to the top left side please, like was in FF 3.6.x Navigation buttons on left, refresh on middle, home button on the right side? Totally absurd! Very uncomfortable. For example if I want to set a new "Home Page" it was so easy to drop the page favicon to the Home Page button what was in FF 3.6.X just 1 cm away to left, but now in FF 4 if I want to do the same I have to grab the favicon throght the whole screen. Yeah, I know its easy to customize the toolbar , but it would be better if all buttons would be the most logical place by default. Thanks!
Comment 22•13 years ago
|
||
Verified fixed on Mozilla/5.0 (Windows NT 6.1; rv:2.2a1pre) Gecko/20110405 Firefox/4.2a1pre
Status: RESOLVED → VERIFIED
Comment 23•13 years ago
|
||
Home Button needs to be on the Left hand side of URL Bar by Default. No need to frustrate non technical users by changing location to right hand side. Non technical users will not know that it is customizable, View ==> Toolbars ==> Customize, as it is not apparent unless you have read somewhere (instructions). That you can drag and drop other stuff i.e. home button, as it is not in the window which opens when you click customize. There are 4,056 Messages when you search "home" in the Beta feedback: http://input.mozilla.com/en-US/beta/search?product=firefox&q=home , and most are confused why it was moved to the right.
You need to log in
before you can comment on or make changes to this bug.
Description
•