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

Frame button interaction changes #1280

Merged
merged 13 commits into from
Feb 15, 2021

Conversation

HerrEmil
Copy link
Contributor

This PR contains a handful of UI and interaction changes to the frame buttons;

The color of the save as favorite button and the options in the download dropdown have been changed.
Current colors:
Screenshot 2021-02-10 at 16 29 08
New colors:
Screenshot 2021-02-10 at 16 29 36

The save as project file option is displayed as an option in the download dropdown instead of as an icon:
Screenshot 2021-02-10 at 16 35 47

The run button is replaced by a stop button while a query is running. The cancel view inside the frame also has two states to differentiate the query cancelling and query cancelled states:
stop query 2

When closing a frame that has been used more than once, a confirmation modal is displayed asking for confirmation and giving the option to cancel the close frame action:
are you sure 3

A demo of the changes is available here: http://frame-button-interactions.surge.sh/

@HerrEmil HerrEmil force-pushed the frame-button-interactions branch from 5bb3659 to 2b20008 Compare February 11, 2021 09:18
@shkirando shkirando self-requested a review February 11, 2021 12:22
@OskarDamkjaer OskarDamkjaer self-requested a review February 15, 2021 07:15
Copy link
Contributor

@OskarDamkjaer OskarDamkjaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@HerrEmil HerrEmil merged commit 3c957fd into neo4j:master Feb 15, 2021
@HerrEmil HerrEmil deleted the frame-button-interactions branch February 15, 2021 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants