Skip to content

Maggie li yd/2487 overlapping charts improvements#2864

Draft
maggie-li-yd wants to merge 3 commits into
developfrom
maggie-li-yd/2487-overlapping-charts-improvements
Draft

Maggie li yd/2487 overlapping charts improvements#2864
maggie-li-yd wants to merge 3 commits into
developfrom
maggie-li-yd/2487-overlapping-charts-improvements

Conversation

@maggie-li-yd

@maggie-li-yd maggie-li-yd commented Jul 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
X1 X2 Action
empty empty disable X2
empty categorical relocate X2 → X1, clear+disable X2
empty continuous relocate X2 → X1, clear+disable X2
categorical empty disable X2 (X1 stays)
categorical categorical clear+disable X2, keep X1 ← fixed
categorical continuous relocate X2 → X1, clear+disable X2
continuous empty disable X2 (X1 stays)
continuous categorical clear+disable X2 (X1 stays)
continuous continuous clear+disable X2 (X1 stays)

Merge Checklist

Please cross check this list if additions / modifications needs to be done on top of your core changes and tick them off. Reviewer can as well glance through and help the developer if something is missed out.

  • Automated Tests (Jasmine integration tests, Unit tests, and/or Performance tests)
  • Updated Manual tests / Demo Config
  • Documentation (Application guide, Admin guide, Markdown, Readme and/or Wiki)
  • Verified that local development environment is working with latest changes (integrated with latest develop branch)
  • following best practices in code review doc

@maggie-li-yd maggie-li-yd linked an issue Jul 7, 2026 that may be closed by this pull request
Overlapping charts improvements #2487
Open
7 tasks
@maggie-li-yd maggie-li-yd force-pushed the maggie-li-yd/2487-overlapping-charts-improvements branch from 0b19cb2 to 4acaab8 Compare July 7, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brandantck brandantck Awaiting requested review from brandantck brandantck will be requested when the pull request is marked ready for review brandantck is a code owner
@jerome-ng jerome-ng Awaiting requested review from jerome-ng jerome-ng will be requested when the pull request is marked ready for review jerome-ng is a code owner
@LSriragavan LSriragavan Awaiting requested review from LSriragavan LSriragavan will be requested when the pull request is marked ready for review LSriragavan is a code owner
@khairul-syazwan khairul-syazwan Awaiting requested review from khairul-syazwan khairul-syazwan will be requested when the pull request is marked ready for review khairul-syazwan is a code owner
@p-hoffmann p-hoffmann Awaiting requested review from p-hoffmann p-hoffmann will be requested when the pull request is marked ready for review p-hoffmann is a code owner
Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Overlapping charts improvements

1 participant

@maggie-li-yd