Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show on App Launch: Use string resources for settings #5549

Conversation

mikescamell
Copy link
Contributor

@mikescamell mikescamell commented Jan 27, 2025

Task/Issue URL: https://app.asana.com/0/1202552961248957/1209246799465881/f

Description

Replaced hardcoded strings with string resources in the app launch settings screen for better localization support. Updated text references for "Last Opened Tab," "New Tab Page," and "Specific Page" options.

Steps to test this PR

String Resources Implementation

  • Navigate to Show on App launch settings
  • Verify "Last Opened Tab" displays correctly from string resources
  • Verify "New Tab Page" displays correctly from string resources
  • Verify "Specific Page" displays correctly from string resources
  • Test in different locales to ensure proper translation support

UI changes

Before After
Screenshot_20250127_160409 Screenshot_20250127_161411

@mikescamell mikescamell marked this pull request as ready for review January 27, 2025 17:11
@anikiki anikiki self-assigned this Jan 27, 2025
Copy link
Contributor

@anikiki anikiki left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mikescamell mikescamell merged commit 1b6e964 into develop Jan 27, 2025
16 checks passed
@mikescamell mikescamell deleted the feature/mike/show-on-app-launch/fix-missing-string-resources branch January 27, 2025 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants