fix: show 'Action Configuration' label in button group item settings

[devtools-boxfusion]

Summary by CodeRabbit

• Bug Fixes
  • The Action Configuration label is now visible in the button group configurator interface.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A single UI configuration property is modified in the button group configurator component. The Action Configuration item label visibility is toggled from hidden to visible by changing the hideLabel property value.

Changes

Cohort / File(s)	Change Summary
Button Group Configurator UI shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts	Changed hideLabel property from true to false for Action Configuration item, making the label visible in the UI

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• duplicate label bug on action cofig #4229: Also modifies label visibility for action-related UI configurations; related UI refinement to action controls.

Suggested reviewers

• James-Baloyi

Poem

🐰 A label once hidden in shadow's embrace,
Now reveals itself in the configurator's space,
A small toggle brings clarity to light,
Where Action Configuration shows what's right! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: making the 'Action Configuration' label visible in button group item settings by changing hideLabel from true to false.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6993b0d and 73454c1.

📒 Files selected for processing (1)

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts (1 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (CodeRabbitReview-Project-Specific-Guidelines.md)

**/*.{ts,tsx}: Eliminate the any type; use unknown type instead for values with unknown types, forcing explicit type checking
Prefer type guards over type casting for type checking
Avoid monolithic types; use discriminated unions with a discriminator property instead
Leverage TypeScript to its full potential as a type system, not merely as a linter

Files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

🧠 Learnings (5)

📚 Learning: 2025-11-20T14:51:46.152Z

Learnt from: AlexStepantsov
Repo: shesha-io/shesha-framework PR: 4235
File: shesha-reactjs/src/components/modelConfigurator/propertiesEditor/provider/reducer.ts:149-149
Timestamp: 2025-11-20T14:51:46.152Z
Learning: In shesha-reactjs model configurator reducer (src/components/modelConfigurator/propertiesEditor/provider/reducer.ts), when handling ArrayFormats.childObjects in the UpdateItem action, itemsType.dataFormat should be set to undefined (not ObjectFormats.object). This is the correct logic for child object array configuration.

Applied to files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

📚 Learning: 2025-06-26T19:59:22.872Z

Learnt from: teboho
Repo: shesha-io/shesha-framework PR: 3461
File: shesha-reactjs/src/designer-components/charts/settingsFormIndividual.ts:1280-1280
Timestamp: 2025-06-26T19:59:22.872Z
Learning: In shesha-reactjs chart settings forms, the styling panels (border, shadow, background) should not include buttonType-based hidden conditions since charts are not button components. The border configuration should follow the standard component pattern without button-specific restrictions.

Applied to files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

📚 Learning: 2025-06-26T20:01:48.838Z

Learnt from: teboho
Repo: shesha-io/shesha-framework PR: 3461
File: shesha-reactjs/src/designer-components/charts/settingsFormIndividual.ts:171-192
Timestamp: 2025-06-26T20:01:48.838Z
Learning: In shesha-reactjs chart settings forms, duplicate input configurations may be intentionally included as part of UI/UX experiments and should be flagged for review during UI/UX review phases rather than immediately removed as errors.

Applied to files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

📚 Learning: 2025-06-12T16:55:57.638Z

Learnt from: teboho
Repo: shesha-io/shesha-framework PR: 3397
File: shesha-reactjs/src/designer-components/charts/bar.tsx:49-52
Timestamp: 2025-06-12T16:55:57.638Z
Learning: For the chart components’ migrators (e.g., BarChartComponent in shesha-reactjs/src/designer-components/charts/bar.tsx), the version 5 step intentionally spreads `{ ...prev, ...defaultConfigFiller }` so that values from `defaultConfigFiller` override any existing properties in `prev`. This reset to new defaults is by design and should not be flagged as an issue.

Applied to files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

📚 Learning: 2025-11-05T09:03:47.960Z

Learnt from: Lihlu
Repo: shesha-io/shesha-framework PR: 4132
File: shesha-reactjs/src/components/calendar/configurable-actions/refresh-calendar.tsx:5-18
Timestamp: 2025-11-05T09:03:47.960Z
Learning: In shesha-reactjs calendar configurable actions (src/components/calendar/configurable-actions/refresh-calendar.tsx), the useRefreshCalendarAction hook registers a configurable action with an empty dependency array []. Do NOT add setRefreshTrigger to the dependency array, as it is not stable (re-created when state.refreshTrigger changes) and would cause a registration loop. The executer uses a functional update pattern setRefreshTrigger((prev) => prev + 1) which doesn't need the current closure, so the action only needs to be registered once on mount.
<!--

Applied to files:

• shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-attempt

🔇 Additional comments (1)

shesha-reactjs/src/components/buttonGroupConfigurator/itemSettings.ts (1)

178-185: Action Configuration label visibility change looks correct

Switching hideLabel to false for the actionConfiguration configurator will surface the existing “Action Configuration” label without affecting validation or surrounding visibility logic. This is a safe, localized UX fix aligned with the PR objective.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}