-
Notifications
You must be signed in to change notification settings - Fork 2
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
card UX improvement #379
base: master
Are you sure you want to change the base?
card UX improvement #379
Conversation
removed unused styles
border-radius: 5px; | ||
} |
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.
In scss you can also do &:hover
within the .name
block.
This is definitely an improvement over what we have now but I think it might be better to have the entire card become highlighted. I think that having the individual fields be highlighted implies that you will only be editing that field. Thoughts? |
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.
Nice
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.
Can you add this to the other cards? It would also be cool if the field you clicked on was what was autoFocus
ed on if it isn't too difficult.
Description
Added a little indication that you can edit takeaways and action items
Type of change
How Has This Been Tested?
took 5 seconds to make sure it worked
Checklist: