Closed Bug 415497 Opened 17 years ago Closed 16 years ago

Put the back button on the window edge to make it an easier target

Categories

(Firefox :: Theme, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3 beta4

People

(Reporter: ventnor.bugzilla, Assigned: ventnor.bugzilla)

Details

(Keywords: access)

Attachments

(1 file, 1 obsolete file)

Attached patch Proof of concept (obsolete) — Splinter Review
Quoth Wikipedia on Fitts' Law:

"Edges (e.g. the menubar in Mac OS) and corners of the computer display (e.g. "Start" button in Windows XP) are particularly easy to acquire because the pointer remains at the screen edge regardless of how much further the mouse is moved, thus can be considered as having infinite width."

When the window is maximized, the back button is 1 or 2 pixels from a position where the cursor on the edge of the screen can target it. The back button is a very common target so I think it would help improve our goal of Flow if we put the back button on the very edge of the window, this giving it infinite width and making it an easier target.

The keyhole might make this impossible on other platforms, but we're using a native-ish interface on Linux. We did this for the scrollbar so we should do this for the back button.

Fire up the VM Roc and see if you agree with this :-)
Attachment #301200 - Flags: superreview?(roc)
Attachment #301200 - Flags: review?(roc)
(see also bug 414842, for windows)
Keywords: access
Attached patch PatchSplinter Review
I actually think the best course of action here is to remove all horizontal padding on toolbars by default. You don't really notice this padding, and it makes for a much smoother experience when you know that putting the cursor against the edge of the screen will click that left-most button or menu.
Assignee: nobody → ventnor.bugzilla
Attachment #301200 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #301371 - Flags: superreview?(roc)
Attachment #301371 - Flags: review?(roc)
Attachment #301200 - Flags: superreview?(roc)
Attachment #301200 - Flags: review?(roc)
Comment on attachment 301371 [details] [diff] [review]
Patch

Can you request review from some UI person on this?
Attachment #301371 - Flags: superreview?(roc)
Attachment #301371 - Flags: superreview+
Attachment #301371 - Flags: review?(roc)
Attachment #301371 - Flags: review?
Attachment #301371 - Flags: review? → review?(gavin.sharp)
Attachment #301371 - Flags: review?(gavin.sharp) → review+
Comment on attachment 301371 [details] [diff] [review]
Patch

Very minor code change, but allows easier targetting of buttons on the toolbar. This has no significant impact on the UI.
Attachment #301371 - Flags: approval1.9?
Comment on attachment 301371 [details] [diff] [review]
Patch

a1.9+=damons
Attachment #301371 - Flags: approval1.9? → approval1.9+
Keywords: checkin-needed
Checking in toolkit/themes/gnomestripe/global/toolbar.css;
/cvsroot/mozilla/toolkit/themes/gnomestripe/global/toolbar.css,v  <--  toolbar.css
new revision: 1.13; previous revision: 1.12
done
Checking in widget/src/gtk2/gtk2drawing.c;
/cvsroot/mozilla/widget/src/gtk2/gtk2drawing.c,v  <--  gtk2drawing.c
new revision: 1.76; previous revision: 1.75
done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: