499204 - tweak default UI for vertically-challenged devices

Status: VERIFIED FIXED

(Firefox :: General, defect, P2)

Description

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

We need a patch to tweak the default UI for vertically-challenged devices; I've no idea where these changes live.

The changes should be:

  - bookmarks toolbar hidden by default
  - statusbar hidden by default
  - toolbar uses small icons by default

Comment 1

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Attachment: Patch v0.1

This could probably be cleaned up & we can probably just set the iconsize on the toolbox & take out the places where the iconsize is explicitly set. Dao would know better.

Unless I missed something this makes sure all the checkboxes match up with what the user sees.

Comment 2

[Dão Gottwald [:dao]]

(In reply to comment #0)
> The changes should be:
> 
>   - bookmarks toolbar hidden by default
>   - statusbar hidden by default
>   - toolbar uses small icons by default

How about hiding the tab bar when there's only one tab (browser.tabs.autoHide)?
Also note that bug 456535 is fixed on trunk. But hiding the menu bar by default would be a usability loss unless the most important actions are available as a toolbar button (à la IE8, Chrome, Safari).

Comment 3

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

(In reply to comment #2)
> How about hiding the tab bar when there's only one tab (browser.tabs.autoHide)?

I like this idea.

> Also note that bug 456535 is fixed on trunk. But hiding the menu bar by default
> would be a usability loss unless the most important actions are available as a
> toolbar button (à la IE8, Chrome, Safari).

Another good idea. I agree that if we do it we would want those things available via toolbar. I would think it might even be OK to have that button open a menu with all of those toolbar items instead of trying to pick the "important" menus.

Comment 4

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Attachment: Patch v0.2

Adds #ifdefs as appropriate (is there a preferred #ifdef vs #ifndef usage?)

* hides statusbar
* hides personal toolbar (bookmarks)
* uses small icons
* sets browser.tabs.autoHide to true

Comment 5

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Attachment: Patch v0.3

Adds throbber to navigator bar

Comment 6

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

pushed http://hg.mozilla.org/mozilla-central/rev/b24ba744b963

This is another case where we'll likely want some followup work done. But probably best done in another bug.

Comment 7

[Paul O'Shannessy [:zpao] (not bugmail, email directly)]

Attachment: Patch v1.0 (as checked in)

The only difference was moving the throbber immediately after the search field so that it's in the right place when in fullscreen mode.

Comment 8

[Tony Chung [:tchung]]

Verified fix on Mozilla/5.0 (Windows; U; WindowsCE 6.0; en-US; rv:1.9.2a2pre)
Gecko/20090810 Minefield/3.6a2pre
*  hides statusbar
* hides personal toolbar (bookmarks)
* uses small icons
* sets browser.tabs.autoHide to true
* Throbber placed after the search bar