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

fix: adjust z-index so Show more/fewer dashboards tooltip is visible [aDHIS2-11647] #1912

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Aug 24, 2021

https://jira.dhis2.org/browse/DHIS2-11647

Because of competing z-indexes on the component cover and tooltip, the tooltip was hidden when the dashboards bar was expanded.

Components should be stacked from bottom to top: cover, dashboards bar, tooltip.
Previously, the z-index of these was: 2000, 2001, 2000 which means the tooltip wasn't visible.
With the new code, the z-index is: 1998, 1999, 2000. (Cannot use ui ComponentCover since z-index is not configurable).

Regular screen:

tooltip-big-screen.mov

Small screen:

tooltip-small-screen-2.mov

@dhis2-bot
Copy link
Contributor

🚀 Deployed on https://pr-1912--dhis2-dashboard.netlify.app

Copy link
Collaborator

@janhenrikoverland janhenrikoverland left a comment

Choose a reason for hiding this comment

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

LGTM

@jenniferarnesen jenniferarnesen merged commit cce97f9 into master Aug 25, 2021
@jenniferarnesen jenniferarnesen deleted the fix/show-fewer-dashboards-tooltip branch August 25, 2021 08:39
dhis2-bot added a commit that referenced this pull request Aug 25, 2021
## [31.17.7](v31.17.6...v31.17.7) (2021-08-25)

### Bug Fixes

* adjust z-index so tooltip on Show more/fewer dashboards button shows ([#1912](#1912)) ([cce97f9](cce97f9))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 31.17.7 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants