-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Convert the password prompt to a class #7219
Convert the password prompt to a class #7219
Conversation
|
||
this.passwordCancel.addEventListener('click', this.close.bind(this)); | ||
this.callback = null; |
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.
Hmm, callback
seems a bit too generic to me. I think that the reason for the current code using updatePassword
is that it's called that in the API (see https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L1361). How about e.g. updateCallback
instead?
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.
You're right. I have changed this to updateCallback
in the new commit.
7b936cd
to
70de173
Compare
Furthermore we introduce two new methods named `setCallback` and `setReason` so external code does not change the properties of the class directly. Finally we update various names of properties and methods to be more self-explanatory.
70de173
to
2b7137b
Compare
/botio-linux preview |
From: Bot.io (Linux)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/2c7d5b62cbbb363/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/2c7d5b62cbbb363/output.txt Total script time: 1.01 mins Published |
Thank you for the patch! |
Furthermore we introduce a new method named
setUpdateCallback
so external code does not change the properties of the class directly. We also update various names of properties and methods to be more self-explanatory.Use, for example, http://async5.org/moz/passwordOU.pdf to verify this patch.