New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding access to page buttons on PrintPreviewDialog by keyboard shortcut keys #3929
Adding access to page buttons on PrintPreviewDialog by keyboard shortcut keys #3929
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3929 +/- ##
====================================================
- Coverage 67.59941% 36.44908% -31.15034%
====================================================
Files 1415 933 -482
Lines 505784 250363 -255421
Branches 41079 36785 -4294
====================================================
- Hits 341907 91255 -250652
+ Misses 157924 153876 -4048
+ Partials 5953 5232 -721
Flags with carried forward coverage won't be shown. Click here to find out more. |
</data> | ||
<data name="sixpagesToolStripButton.Text"> | ||
<value xml:space="preserve">Six pages</value> | ||
<value xml:space="preserve">Six pages (Ctrl+5)</value> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was the reason to go with Ctrl+5
and not with Ctrl+6
? Did you find any guidance about it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was my suggestion #3682 (comment) that everyone has agreed with.
In my opinion it is a logical sequence - option 1, 2, ...5, and not based on number of pages. E.g. if there was a sixth option to show 12 pages - as a user I wouldn't expect CTRL+12 but CTRL+6.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had you guys looked at what Office or windows are doing in similar cases?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
src/System.Windows.Forms/src/System/Windows/Forms/Printing/PrintPreviewDialog.cs
Outdated
Show resolved
Hide resolved
a589c8f
to
08a78c7
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Fixes #3682
Proposed changes
Customer Impact
Regression?
Risk
Test methodology
Test environment(s)
Microsoft Reviewers: Open in CodeFlow