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

[WIP] Feature/multi drag dimensions #206

Merged
merged 32 commits into from
Feb 27, 2019

Conversation

jenniferarnesen
Copy link
Collaborator

@jenniferarnesen jenniferarnesen commented Jan 31, 2019

Feature: enable user to sort the selected items list by dragging multiple items at the same time. This feature should be available in the Dynamic Dimensions, Data Dimension, and Period Dimension dialogs. In order to share code/remove duplication, since Period Dimension is a d2-ui component, it was necessary to separate out the common functionality into a new shared component "Item Selector". This shared component is currently home in the data-visualizer-app's new "shared" package. This may change as we evolve the structure of the repositories.

Changes included in this PR:

  • Create separate package shared that can contain common modules and components
  • Factor out the ItemSelector component and put it in shared
    -- note that the ItemSelector component will be used by d2-ui PeriodSelector
  • Implement multi-drag functionality in ItemSelector based on the react-beautiful-dnd package
  • Move the colors module to shared
  • Change DialogManager so that only the orgunit dimension component is retained in the DOM after the dialog is closed. This was to fix an issue where the drag/drop left the data and dynamic dimension selected items in an invalid state when dropping outside of the dialog. (the reason for keeping orgunit in the DOM is so that all the org unit data doesn't have to be refetched - lots of queries. There is a separate discussion on how to do this better than showing/hiding components)

Fixes [DHIS2-5480]

@janhenrikoverland
Copy link
Collaborator

This feature has been smoke tested and we will merge it for the sake of getting started on the new mono repo. Needs more testing before being released or added to d2-ui.

@janhenrikoverland janhenrikoverland merged commit 7744517 into master Feb 27, 2019
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 33.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 32.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 33.1.13 🎉

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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants