This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 229106 - Patch for: Create branch dialog: clash of mnemonics and other usability issues
Summary: Patch for: Create branch dialog: clash of mnemonics and other usability issues
Status: VERIFIED FIXED
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: Git (show other bugs)
Version: 7.4
Hardware: PC Windows 7
: P3 normal (vote)
Assignee: Ondrej Vrabec
URL:
Keywords: PATCH_AVAILABLE
Depends on:
Blocks:
 
Reported: 2013-04-27 15:00 UTC by markiewb
Modified: 2013-07-23 22:02 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
shows the issue (70.45 KB, image/png)
2013-04-27 15:00 UTC, markiewb
Details
Proposed patch (12.09 KB, patch)
2013-04-27 15:45 UTC, markiewb
Details | Diff
Patch in action (8.33 KB, image/png)
2013-04-27 15:51 UTC, markiewb
Details
Proposed patch v2 (11.50 KB, patch)
2013-04-28 21:34 UTC, markiewb
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description markiewb 2013-04-27 15:00:48 UTC
Created attachment 133872 [details]
shows the issue

ACTUAL 
Clash of mnemonics: "&Create" vs. "&Checkout..."

EXPECTED
no clash

FYI: was introduced by http://hg.netbeans.org/core-main/rev/d5e0b056150
Comment 1 markiewb 2013-04-27 15:42:59 UTC
Other subtle issues/inconsistences in the dialog: 
* Why there are mnemonics for "Commit &ID", "&Author", "&Message"? The user can press the mnemonics, but the fields next to the labels are always disabled/readonly. So I propose to remove the mnemonics
* some labels end with a colon - some not
* some labels are not aligned
Comment 2 markiewb 2013-04-27 15:45:12 UTC
Created attachment 133874 [details]
Proposed patch

I like to propose the following patch to solve the described issues.
* Updated mnemonics
* Removed mnemonics for labels, which are linked to readonly/disabled controls
* Some minor layout modifications, so it looks cleaner

Please have a look.
Comment 3 markiewb 2013-04-27 15:51:12 UTC
Created attachment 133875 [details]
Patch in action
Comment 4 markiewb 2013-04-27 16:26:19 UTC
(In reply to comment #1)
> * Why there are mnemonics for "Commit &ID", "&Author", "&Message"? The user can
> press the mnemonics, but the fields next to the labels are always
> disabled/readonly. So I propose to remove the mnemonics

I looked it up. The issue also applies to HG implementation. Should I file a separate issue for this?
Comment 5 Ondrej Vrabec 2013-04-28 16:02:32 UTC
(In reply to comment #1)
> * Why there are mnemonics for "Commit &ID", "&Author", "&Message"? The user can
> press the mnemonics, but the fields next to the labels are always
> disabled/readonly. So I propose to remove the mnemonics
Well, they're control fields so i assumed they should have a mnemonic. User can navigate via keyboard and still with keyboard for example select and copy the field's value. I have no idea if A11Y specs says anything about this, do you? In order to avoid any accessibility issues, i would rather keep the mnemonics. Any idea?
Comment 6 markiewb 2013-04-28 20:19:29 UTC
(In reply to comment #5)
> (In reply to comment #1)
> > * Why there are mnemonics for "Commit &ID", "&Author", "&Message"? The user can
> > press the mnemonics, but the fields next to the labels are always
> > disabled/readonly. So I propose to remove the mnemonics
> Well, they're control fields so i assumed they should have a mnemonic. User can
> navigate via keyboard and still with keyboard for example select and copy the
> field's value. 
You are right. I wasn't aware that you still can select the content in readonly controls. I will update the patch.
Comment 7 markiewb 2013-04-28 21:34:27 UTC
Created attachment 133897 [details]
Proposed patch v2

Here the patch without the removal of mnemonics of the readonly control's labels
Comment 8 Ondrej Vrabec 2013-04-29 08:33:52 UTC
thanks for the patch, pushed as http://hg.netbeans.org/core-main/rev/d7cf7ee1b780
Comment 9 Quality Engineering 2013-05-02 14:12:19 UTC
Integrated into 'main-golden', will be available in build *201305021042* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/d7cf7ee1b780
User: Benno Markiewicz <markiewb@netbeans.org>
Log: #229106 - Create branch dialog: clash of mnemonics and other usability issues
Comment 10 markiewb 2013-07-23 22:02:39 UTC
Verified in 201307222300