Skip to content

Feature/541#610

Merged
ddelpiano merged 4 commits intodevelopmentfrom
feature/541
Dec 6, 2022
Merged

Feature/541#610
ddelpiano merged 4 commits intodevelopmentfrom
feature/541

Conversation

@aranega
Copy link
Copy Markdown
Member

@aranega aranega commented Nov 22, 2022

Fixes #541

The color picker is removed from the "hover menu" and is integrated directly in each tree item.
Some setIntervals that didn't have the expected behavior have been removed.

Regarding the color update, it follows this behavior:

  • if a color is assigned to a leaf, only the color for the leaf is changing;
  • if a color is assigned to a container node (node with children), all the direct children inherit of the color;
  • if a color is assigned to a leaf and among the sibiling of the leaf, there is one that hasn't the same color, the container node color passes as gray;
  • following this policy, when random colors are given to a container node, the container node color is gray while all children have their dedicated color.

@aranega aranega requested a review from ddelpiano November 22, 2022 19:24
Comment thread webapp/components/general/ControlPanelTreeItem.js Outdated
@ddelpiano ddelpiano self-requested a review November 23, 2022 13:53
@ddelpiano ddelpiano merged commit 045a78f into development Dec 6, 2022
@ddelpiano ddelpiano deleted the feature/541 branch April 14, 2023 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants