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

feat: toolbar UI update with hoverable menu (#1478) #1509

Merged

Conversation

HendrikThePendric
Copy link
Contributor

@HendrikThePendric HendrikThePendric commented Jun 20, 2023

Implements DHIS2-15167

Relates to dhis2/line-listing-app#368 & dhis2/data-visualizer-app#2358


Key features

  1. Adds Toolbar and ToolbarSidebar components to build the toolbar layout
  2. Adds HoverMenubar and related components to build hoverable menu within the toolbar
  3. Adds UpdateButton and InterpretationsAndDetailsToggler to be placed in the toolbar
  4. Updates FileMenu to be a hoverable dropdown menu, leveraging the new hover-menu-bar components.
  5. Adds initiallyActiveTabKey to VisualizationOptions component so it can be opened at a specified tab.

BREAKING CHANGE

The updated FileMenu is not compatible with the previous version. Apps will be required to update their toolbar before using this version of analytics.


TODO

  • Add tests
  • Add demo

Screenshots

Screenshot 2023-06-12 at 18 20 44

Not the most glamorous screenshot, but it works.

@HendrikThePendric HendrikThePendric force-pushed the feat/toolbar-ui-update-with-hoverable-menu-bar branch from 244145d to 24d21c1 Compare June 20, 2023 09:14
@HendrikThePendric HendrikThePendric enabled auto-merge (squash) June 20, 2023 09:18
BREAKING CHANGE
The `FileMenu` is now using the new `HoverMenuBar` components which makes this version of the `FileMenu` incompatible with the previous version. Apps will be need to update their toolbar and file menu before using this version of analytics.
@HendrikThePendric HendrikThePendric force-pushed the feat/toolbar-ui-update-with-hoverable-menu-bar branch from 24d21c1 to 129ad4a Compare June 20, 2023 09:22
@HendrikThePendric HendrikThePendric merged commit 0a51153 into master Jun 20, 2023
@HendrikThePendric HendrikThePendric deleted the feat/toolbar-ui-update-with-hoverable-menu-bar branch June 20, 2023 09:28
dhis2-bot added a commit that referenced this pull request Jun 20, 2023
# [26.0.0](v25.2.3...v26.0.0) (2023-06-20)

### Features

* toolbar UI update with hoverable menu ([#1478](#1478)) ([#1509](#1509)) ([0a51153](0a51153))

### BREAKING CHANGES

* The `FileMenu` is now using the new `HoverMenuBar` components which makes this version of the `FileMenu` incompatible with the previous version. Apps will be need to update their toolbar and file menu before using this version of analytics.
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 26.0.0 🎉

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