Skip to content

[FEATURE] enhance StatChart with text mode options and refactor display logic - #758

Open
Nexucis wants to merge 5 commits into
mainfrom
statchart/add-textmode
Open

[FEATURE] enhance StatChart with text mode options and refactor display logic#758
Nexucis wants to merge 5 commits into
mainfrom
statchart/add-textmode

Conversation

@Nexucis

@Nexucis Nexucis commented Jul 28, 2026

Copy link
Copy Markdown
Member

this PR is the same as #524. I am just solving the merge conflict and I have preserved the commit from @abelyakin.

I am opening this PR in order to be able to merge it as the merge conflict was the last thing to solve before merging it.

abelyakin and others added 4 commits January 16, 2026 03:43
…ogic

- Introduced a new  property in the StatChart schema to control how values and names are displayed.
- Updated the migration logic to map  from panel options and handle various display scenarios.
- Refactored the StatChartBase component to utilize  for determining what to display (value, name, both, or none).
- Adjusted tests and expected outputs to reflect changes in text mode handling.
- Enhanced the StatChartOptionsEditorSettings to allow users to select the desired text mode.

Signed-off-by: Alexander Belyakin <21216343+abelyakin@users.noreply.github.com>
Co-authored-by: Antoine THEBAUD <antoine.thebaud@yahoo.fr>
Signed-off-by: Alexander Belyakin <21216343+abelyakin@users.noreply.github.com>
@Nexucis
Nexucis requested review from a team and AntoineThebaud as code owners July 28, 2026 08:47
@Nexucis
Nexucis requested review from Gladorme and removed request for a team July 28, 2026 08:47
@Nexucis
Nexucis force-pushed the statchart/add-textmode branch 2 times, most recently from d0c838e to a9e5d45 Compare July 29, 2026 09:33
@Nexucis
Nexucis force-pushed the statchart/add-textmode branch from a9e5d45 to 291456b Compare July 29, 2026 09:35
@Nexucis

Nexucis commented Jul 29, 2026

Copy link
Copy Markdown
Member Author

the cue diff does not make sense for me ... I will check later what's going one here

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