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

UI updates based on last meeting with the surgery list #208

Merged
merged 13 commits into from
Feb 27, 2022

Conversation

haihan-lin
Copy link
Collaborator

@haihan-lin haihan-lin commented Feb 25, 2022

updates:

  • Allows overlap list toggle through triangle regardless surgery selection
  • Allows overlap to be selected when the main item is not selected (it will add the main item with the overlap item to the selected).
  • Fix search
  • fix hover to show full text (but kinda hard to see with text overlap)
  • 'Only xxx' is always on top

Copy link
Member

@JackWilb JackWilb left a comment

Choose a reason for hiding this comment

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

Looks good to me. One weird thing I noticed is that if you have only CABG selected and then you try to select all CABG again, it just collapses the menu. I thought that would select all CABG, not remove my selection.

@haihan-lin
Copy link
Collaborator Author

Should fixed it with the new commit.

Also btw the new update only allows selection through clicking text (because we need separate interaction between the triangle expanding list and procedure selection).

@haihan-lin haihan-lin merged commit 7cf23f7 into main Feb 27, 2022
@haihan-lin haihan-lin deleted the exclusive-counts branch February 27, 2022 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants