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

Medium: Hovering over Dashboard Widget Rows #41

Closed
PhilBarrett opened this issue May 30, 2019 · 10 comments
Closed

Medium: Hovering over Dashboard Widget Rows #41

PhilBarrett opened this issue May 30, 2019 · 10 comments

Comments

@PhilBarrett
Copy link
Member

When hovering over the rows within any widgets on the dashboard, no row hover effects are being displayed. It’s convention to invert the colour when hovering over any rows within a panel or widget. The mouse cursor should also change to a pointer (cursor: pointer;)

@PhilBarrett PhilBarrett changed the title Hovering over Dashboard Widget Rows Medium: Hovering over Dashboard Widget Rows May 30, 2019
@BogdanScherban
Copy link
Collaborator

@simongamester

Could you please clarify, what do you mean as Dashboard? Which elements of UI should have cursor "pointer"?
Could you please add screenshot?

@simongamester
Copy link
Collaborator

@BogdanScherban Apologies, I mean Patient Summary. The page were we display widgets...

Screenshot 2019-05-31 at 10 42 09

Cursor 'pointer' should be applied to the table rows within these widgets.

@BogdanScherban
Copy link
Collaborator

OK! I need in 0.2-0.25 h for this.

@kuvakina
Copy link
Collaborator

kuvakina commented Jun 3, 2019

@simongamester
Copy link
Collaborator

@BogdanScherban This is caused because the View button doesn't have a white background by default, I think it's transparent. So when you hover over the row, you're seeing a transition on the button between transparent and white, but the row background changes to green immediately.

Also, please could you keep the green border on the View button when not hovering over it. It looks like the View button has no border in it's default state.

Screenshot 2019-06-03 at 18 09 58

@simongamester
Copy link
Collaborator

@BogdanScherban I've also checked the Patient Summary and the rows within the widgets aren't hovering over in green.

@BogdanScherban
Copy link
Collaborator

@simongamester

OK, should I add hovering to Patient Summary panels?
Because I remember that it was done but later we moved it.

@simongamester
Copy link
Collaborator

The only panels we want to add hovers on the patient summary are the rows within widgets i.e.

Screenshot 2019-06-04 at 14.07.16.png

@BogdanScherban
Copy link
Collaborator

@simongamester
Copy link
Collaborator

@PhilBarrett Looks good to me. Thanks Bogdan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants