Merge lp://qastaging/~jpickett/simple-scan/bug-842603 into lp://qastaging/~simple-scan-team/simple-scan/trunk

Proposed by Joel Pickett
Status: Rejected
Rejected by: Robert Ancell
Proposed branch: lp://qastaging/~jpickett/simple-scan/bug-842603
Merge into: lp://qastaging/~simple-scan-team/simple-scan/trunk
Diff against target: 11 lines (+1/-0)
1 file modified
data/simple-scan.ui (+1/-0)
To merge this branch: bzr merge lp://qastaging/~jpickett/simple-scan/bug-842603
Reviewer Review Type Date Requested Status
Robert Ancell Disapprove
Review via email: mp+82382@code.qastaging.launchpad.net

Description of the change

Change the Document - Preferences option to use the R key accelerator to avoid collision with Print (which also uses P)

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

This is actually adding an accelerator where there wasn't one before. The preferences don't need an accelerator as they're an infrequently used item. The bug linked to this is actually for the mnemonic for the menu item.

review: Disapprove

Unmerged revisions

558. By Joel Pickett

use R accel key for Preferences

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches