Skip to content

Conversation

yousoph
Copy link
Member

@yousoph yousoph commented Jul 31, 2025

Summary

  • Updated capitalization of table chart configuration options in the Customize tab to use proper sentence case
  • Fixes inconsistent capitalization across table configuration options

Changes Made

  • "Show Cell bars""Show cell bars" (lowercase "cell")
  • "add colors to cell bars for +/-""Add colors to cell bars for +/-" (capitalize "Add")
  • "basic conditional formatting""Basic conditional formatting" (capitalize "Basic")
  • "Custom Conditional Formatting""Custom conditional formatting" (lowercase "conditional formatting")

Files Updated

  • superset-frontend/plugins/plugin-chart-table/src/controlPanel.tsx
  • superset-frontend/plugins/plugin-chart-ag-grid-table/src/controlPanel.tsx
  • All translation files (messages.pot and 21 language-specific .po files)

Screenshots

Before:
image

After:

Test plan

  • Verify table chart configuration options display with correct capitalization in the UI
  • Confirm both regular table and AG Grid table charts show updated labels
  • Check that internationalization still works correctly

🤖 Generated with Claude Code

Update capitalization of table chart configuration options in the Customize tab:
- "Show Cell bars" → "Show cell bars"
- "add colors to cell bars for +/-" → "Add colors to cell bars for +/-"
- "basic conditional formatting" → "Basic conditional formatting"
- "Custom Conditional Formatting" → "Custom conditional formatting"

Updates both table and ag-grid-table plugins and all translation files.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@dosubot dosubot bot added the explore:design Related to the Explore UI/UX label Jul 31, 2025
Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've completed my review and didn't find any issues.

Files scanned
File Path Reviewed
superset-frontend/plugins/plugin-chart-ag-grid-table/src/controlPanel.tsx
superset-frontend/plugins/plugin-chart-table/src/controlPanel.tsx

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@sadpandajoe
Copy link
Member

@yousoph @kasiazjc should labels always be sentence case? If so I can probably add it to my PR to do detection going forward

@sadpandajoe sadpandajoe added the AI 🤖 generated by AI label Aug 1, 2025
@yousoph
Copy link
Member Author

yousoph commented Aug 1, 2025

@yousoph @kasiazjc should labels always be sentence case? If so I can probably add it to my PR to do detection going forward

@sadpandajoe yes but there's a handful of exceptions to the rule, @mistercrunch was taking a look at adding something also

@mistercrunch
Copy link
Member

mistercrunch commented Aug 1, 2025

Yeah this PR #34437 would fix all of the non-title-casing in the app (after I get AI to fx the thousands of now-linting error). Seems worthwhile. I'd like to test to see if flask-babel will break the i18n linkage before processing. If it breaks all translations that's no good ...

That concern probably applies to smaller PRs doing this at a smaller scale too.

My vote: go ahead with the big bang case-switching and line up a PR to use AI to translate everything ... All a sudden it's quite a bit more work.

yousoph added a commit that referenced this pull request Aug 1, 2025
- Update msgid entries in all .po/.pot files for the 4 string changes
- Files maintain their original translations, only msgid strings updated
- Follows same pattern as PR #34438
@yousoph yousoph merged commit 7c2ec4c into master Aug 1, 2025
71 of 79 checks passed
@yousoph yousoph deleted the fix/table-chart-label-capitalization branch August 1, 2025 19:02
LisaHusband pushed a commit to LisaHusband/superset that referenced this pull request Aug 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI 🤖 generated by AI explore:design Related to the Explore UI/UX preset-io size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants