-
Notifications
You must be signed in to change notification settings - Fork 10
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
Always require confirmation click when downloading LoginFile #424
Conversation
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 PR does not implement the final design yet.
back-to-download
is still referenced inExport.css
and can be removed.
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.
Simple and effective. Nice 👍
I would've loved to have a transition between the buttons, with the text fading out or smth. But i guess that since we're opening a popup the "jump" is not meant to be seen so we don't really need one.
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.
I agree with Matheo that animating between the two states would be nice.
Note that not always a popup dialogue is opening, specifically if the user has his browser setup to download files immediately.
In this case he'd be seeing the animation.
For the fallback-download
case I would personally not unveil the Continue
text in the continue button as it breaks the layout to a degree.
634d11a
to
edac524
Compare
I created issue #426 for the button transitions, as it's too much work now to figure this out and not worth the delay in releasing this fix. |
Replaces #423.