Skip to content

fix: correct not available on web tooltip - #1828

Open
kp-krish wants to merge 2 commits into
freeCodeCamp:mainfrom
kp-krish:fix/not-available-web-tooltip
Open

fix: correct not available on web tooltip#1828
kp-krish wants to merge 2 commits into
freeCodeCamp:mainfrom
kp-krish:fix/not-available-web-tooltip

Conversation

@kp-krish

@kp-krish kp-krish commented Aug 22, 2026

Copy link
Copy Markdown

Checklist:

Closes #1827

The not_available_web snackbar was missing a separator and read Not available use the web version. This updates it to Not available - use the web version, matching the wording of the neighbouring coming_soon_web string (Coming soon - use the web version).

The string is shown by disabledButtonSnack() in lib/ui/views/learn/landing/landing_viewmodel.dart when a curriculum item is web-only.

The same correction is applied to app_es.arb and app_pt.arb, which both carry the untranslated English string, along with the corresponding generated localization files.

Verified locally on Flutter 3.44.9 (the version pinned in CI):

  • flutter pub get regenerates the localization files with no further diff, so the committed generated output matches gen-l10n exactly
  • flutter analyze --no-fatal-warnings reports no new issues
  • flutter test unit and flutter test services pass

Add the missing separator to the `not_available_web` string so it reads
"Not available - use the web version", matching the existing
`coming_soon_web` string.

Closes freeCodeCamp#1827

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

@Nirajn2311 Nirajn2311 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you manually do the same change for the other languages as well. As of now Crowdin is not used

Mirror the `not_available_web` fix in the Spanish and Portuguese
localizations, which currently carry the untranslated English string.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@kp-krish

Copy link
Copy Markdown
Author

Thanks for the review, and for clarifying that Crowdin isn't in use right now.

I've applied the same correction to app_es.arb and app_pt.arb plus their generated files. Both locales currently hold the untranslated English string for this key (in fact all 300 keys in both files are still identical to the English source), so I mirrored the English wording rather than introducing a Spanish or Portuguese translation for this one string alone. Happy to add real translations instead if you'd prefer.

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.

Typo in tooltip when courses are not available on mobile

2 participants