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

Remove the currentScaleValue getter from PDFViewerApplication (issue 6158) #6182

Merged
merged 1 commit into from
Jul 9, 2015
Merged

Remove the currentScaleValue getter from PDFViewerApplication (issue 6158) #6182

merged 1 commit into from
Jul 9, 2015

Conversation

Snuffleupagus
Copy link
Collaborator

The next step towards fixing issue #6158.

We can just as well access pdfViewer.currentScaleValue directly in PDFViewerApplication, instead of having a helper function which just acts as a wrapper for it.

…sue 6158)

*The next step towards fixing issue 6158.*

We can just as well access `pdfViewer.currentScaleValue` directly in `PDFViewerApplication`, instead of having a helper function which just acts as a wrapper for it.
@Snuffleupagus
Copy link
Collaborator Author

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Jul 7, 2015

From: Bot.io (Linux)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.21.233.14:8877/247e16fc4a29d5c/output.txt

@timvandermeij timvandermeij self-assigned this Jul 7, 2015
timvandermeij added a commit that referenced this pull request Jul 9, 2015
…on-get-currentScaleValue

Remove the `currentScaleValue` getter from `PDFViewerApplication` (issue 6158)
@timvandermeij timvandermeij merged commit ccfafea into mozilla:master Jul 9, 2015
@timvandermeij
Copy link
Contributor

Good, thanks!

@Snuffleupagus Snuffleupagus deleted the remove-PDFViewerApplication-get-currentScaleValue branch July 9, 2015 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants