Skip to content

Comments

fix(Select): fix Russian translations for Select#35751

Open
innovark37 wants to merge 6 commits intoapache:masterfrom
innovark37:innovark/fix/fix-translation-for-select
Open

fix(Select): fix Russian translations for Select#35751
innovark37 wants to merge 6 commits intoapache:masterfrom
innovark37:innovark/fix/fix-translation-for-select

Conversation

@innovark37
Copy link
Contributor

@innovark37 innovark37 commented Oct 20, 2025

User description

SUMMARY

Strings Select all and Deselect all are not extracted from Select component when following the documentation. This PR fixes missing translations for Select component.
FYI: Deselect all is translated only because ListView component uses this string.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE
image
AFTER
image

TESTING INSTRUCTIONS

  1. Add next lines to config file:
BABEL_DEFAULT_LOCALE = "ru"
LANGUAGES = {
  "ru": {"flag": "ru", "name": "Русский"},
  "en": {"flag": "us", "name": "English"}
}
  1. Start the application.
  2. Create dashboard.
  3. Add native filter (Filter type = "Value").
  4. Verify the translations in Select footer (strings Select all and Deselect all).

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

CodeAnt-AI Description

Fix Russian translations and footer wrapping for Select component

What Changed

  • "Select all" and "Deselect all" strings are added to Russian translations so those labels now appear translated in the Select footer
  • Footer action buttons allow wrapping to avoid overflow on narrow layouts, preventing layout truncation of the count labels
  • Counts remain visible next to the translated labels (e.g., "Выбрать все (5)")

Impact

✅ Clearer Russian Select labels
✅ No truncated footer actions on small screens
✅ Consistent translated label + count display

💡 Usage Guide

Checking Your Pull Request

Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.

Talking to CodeAnt AI

Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.

Example

@codeant-ai ask: Can you suggest a safer alternative to storing this secret?

Preserve Org Learnings with CodeAnt

You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:

@codeant-ai: Your feedback here

This helps CodeAnt AI learn and adapt to your team's coding style and standards.

Example

@codeant-ai: Do not flag unused imports.

Retrigger review

Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

Check Your Repository Health

To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

@github-actions github-actions bot added i18n Namespace | Anything related to localization i18n:russian Translation related to Russian language packages labels Oct 20, 2025
@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.25%. Comparing base (54eb631) to head (5d8c43c).

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #35751       +/-   ##
===========================================
+ Coverage        0   68.25%   +68.25%     
===========================================
  Files           0      638      +638     
  Lines           0    47524    +47524     
  Branches        0     5181     +5181     
===========================================
+ Hits            0    32439    +32439     
- Misses          0    13805    +13805     
- Partials        0     1280     +1280     
Flag Coverage Δ
hive 43.14% <ø> (?)
mysql 66.25% <ø> (?)
postgres 66.30% <ø> (?)
presto 46.74% <ø> (?)
python 68.22% <ø> (?)
sqlite 66.02% <ø> (?)
unit 100.00% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sadpandajoe sadpandajoe self-requested a review October 21, 2025 17:24
…r-select

# Conflicts:
#	superset/translations/ru/LC_MESSAGES/messages.po
…r-select

# Conflicts:
#	superset/translations/ru/LC_MESSAGES/messages.po
@codeant-ai-for-open-source
Copy link
Contributor

CodeAnt AI is reviewing your PR.


Thanks for using CodeAnt! 🎉

We're free for open-source projects. if you're enjoying it, help us grow by sharing.

Share on X ·
Reddit ·
LinkedIn

@codeant-ai-for-open-source codeant-ai-for-open-source bot added the size:XS This PR changes 0-9 lines, ignoring generated files label Dec 23, 2025
@codeant-ai-for-open-source
Copy link
Contributor

Nitpicks 🔍

🔒 No security issues identified
⚡ Recommended areas for review

  • Accessibility
    The new buttons render the count inline but do not expose an explicit aria-label that includes the dynamic count. Screen reader users may not get the full context (action + number). Verify the buttons are announced correctly and consider providing aria-labels that include the count or an aria-live region for updates.

  • i18n / pluralization
    The change separates the static label from the dynamic count (e.g. {t('Select all')} {(${bulkSelectCounts.selectable})}). While this enables extraction of the base strings, it may break correct word order or pluralization rules in languages such as Russian where the count's placement and inflection matter. Consider using a single localized message with a placeholder or pluralization support so translators can place the count appropriately.

  • Layout change
    The bulk actions container gained the wrap prop (StyledBulkActionsContainer justify="center" wrap). Confirm that the styled component supports the wrap prop and that the visual change does not negatively affect compact layouts or overflow in small viewports.

@codeant-ai-for-open-source
Copy link
Contributor

CodeAnt AI finished reviewing your PR.

@bito-code-review
Copy link
Contributor

bito-code-review bot commented Dec 23, 2025

Code Review Agent Run #ed0c54

Actionable Suggestions - 0
Additional Suggestions - 1
  • superset-frontend/packages/superset-ui-core/src/components/Select/Select.tsx - 1
    • Valid UI Prop Addition · Line 495-495
      The addition of the 'wrap' prop to StyledBulkActionsContainer is valid, as it corresponds to a supported boolean prop in the underlying Antd Flex component (used via the custom Flex wrapper). This allows flex items (the Select all and Deselect all buttons) to wrap to the next line if the container width is insufficient, improving layout responsiveness without altering existing behavior.
Review Details
  • Files reviewed - 2 · Commit Range: 8846250..5d8c43c
    • superset-frontend/packages/superset-ui-core/src/components/Select/Select.tsx
    • superset/translations/ru/LC_MESSAGES/messages.po
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

AI Code Review powered by Bito Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n:russian Translation related to Russian language i18n Namespace | Anything related to localization packages size/XS size:XS This PR changes 0-9 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants