Skip to content
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

Update print preview when switching printers #294

Merged
merged 1 commit into from
Mar 4, 2016
Merged

Update print preview when switching printers #294

merged 1 commit into from
Mar 4, 2016

Conversation

mtaron
Copy link

@mtaron mtaron commented Mar 4, 2016

If you switch printers the print preview doesn't update until you change
another property like Orientation. This is noticeable when switch from
something like print to PDF which has no margins to a physical printer
that does. (bug #292)

Fix is to listen to OptionChanged and invalidate the preview when an
event with a null OptionId is raised.

If you switch printers the print preview doesn't update until you change
another property like Orientation. This is noticeable when switch from
something like print to PDF which has no margins to a physical printer
that does. (bug #292)

Fix is to listen to OptionChanged and invalidate the preview when an
event with a null OptionId is raised.
@azurecla
Copy link

azurecla commented Mar 4, 2016

Hi @mtaron, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@clandrew
Copy link
Contributor

clandrew commented Mar 4, 2016

I took a look, as well. Looks good to me. Good catch!

@mtaron
Copy link
Author

mtaron commented Mar 4, 2016

Thanks! I don't have write access so one of you guys will need to merge the pull request.

damyanp added a commit that referenced this pull request Mar 4, 2016
Update print preview when switching printers
@damyanp damyanp merged commit ba3e4aa into microsoft:master Mar 4, 2016
@mtaron mtaron deleted the updateprinter branch March 4, 2016 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants