-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Feature: Show modal when extracting password protected archives #10413
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
Feature: Show modal when extracting password protected archives #10413
Conversation
Do you think we can reuse the existing "extract to" dialog and add the logic for this? |
I think I can add the password field to the dialog and make it visible only when the archive is encrypted. Is that ok? |
I think that's acceptable, we'll probably add additional options in the future so having it one dialog will be helpful. |
Now when the archive is encrypted it looks like this |
I'd expect the other options to only show if you click "Extract To" |
It already works like that |
I think we'll need to make a couple of changes to the dialog to make it more generic
|
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.
Looks good UI wise.
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.
LGTM
Resolved / Related Issues
Items resolved / related issues by this PR.
Change List
destinatioFolder
to prevent the folder from being created even when the user pressesCancel
Extract
buttonOthers
7z
archives, but it also exists in the release version (I'm opening a different issue to track that)Comments
Note that with the
SevenZip
library we can pass the password only in the constructorValidation
How did you test these changes?