-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Made calculation options hidden when empty: #1076
Conversation
@asaadmahmood can you run this and verify the UX? |
@harshilsharma63 For this one, we want to show all the calculate dropdowns if a person is hovering over the row. |
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.
See comment. Also we can make the Calculate option by default a bit greyed (opacity .6) but if it has a value, we bump up the opacity to default (1).
@asaadmahmood can you take a look at this again? I've made the required changes. |
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.
The hover doesn't work on the plugin.
…calboard into table-calculation-defaults
@asaadmahmood please take a look again. I've fixed the issue with plugin mode with Mattermost v5.38 and the latest |
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
* Made calculation options hidden when empty: * WIP * Made calculation row visible on hover * Updated tests * Fixed issue with plugin mode * Fixed a bug where options didn't hide after selecting an option * Updating opacity Co-authored-by: Asaad Mahmood <asaadmahmood@users.noreply.github.com>
Summary
Displaying table calculation options only when not empty and on hover.
Ticket Link
Fixes #960