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: display info msg when no legend is found (DHIS2-147) #656

Merged
merged 2 commits into from
Oct 21, 2020

Conversation

martinkrulltott
Copy link
Contributor

Implements DHIS2-147

Relates to dhis2/data-visualizer-app#1355


Key features

  1. Info message for no legend

Description

When the legend option “Use pre-defined legend per data item” is used, each series item is assigned their own legend. In case the series item has no legend assigned to it, the hover tooltip will no display a correct info message to inform the user why no legend is applied.


Screenshots

previously it was incorrectly showing a an empty row with just a : character when no legend was found
image

no legend now shows a proper info message
image

items with legend still show their regular tooltip as before
image

@martinkrulltott martinkrulltott changed the title fix: display info msg when no legend is found fix: display info msg when no legend is found (DHIS2-147) Oct 21, 2020
Copy link
Member

@edoardo edoardo left a comment

Choose a reason for hiding this comment

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

👍

@martinkrulltott martinkrulltott merged commit 19d7860 into master Oct 21, 2020
@martinkrulltott martinkrulltott deleted the fix/DHIS2-147-charts-legend-set-v2 branch October 21, 2020 09:56
dhis2-bot added a commit that referenced this pull request Oct 21, 2020
## [11.1.1](v11.1.0...v11.1.1) (2020-10-21)

### Bug Fixes

* display info msg when no legend is found (DHIS2-147) ([#656](#656)) ([19d7860](19d7860))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 11.1.1 🎉

The release is available on:

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