Skip to content

Localize DisplayAlert dialogs and standardize dialog buttons#92

Merged
phoneuooyd merged 1 commit intomainfrom
codex/refactor-display-alerts-for-localization
Mar 6, 2026
Merged

Localize DisplayAlert dialogs and standardize dialog buttons#92
phoneuooyd merged 1 commit intomainfrom
codex/refactor-display-alerts-for-localization

Conversation

@phoneuooyd
Copy link
Copy Markdown
Owner

Motivation

  • Usunąć hardcoded teksty w dialogach DisplayAlert(...) i przenieść je do zasobów w celu pełnej lokalizacji i łatwiejszej konserwacji.
  • Ujednolicić etykiety przycisków dialogów przez wspólne klucze zasobów, aby stosować spójne tłumaczenia (OK/Yes/No).

Description

  • Zastąpiono wszystkie wywołania z literalnymi ciągami w HomePage.xaml.cs, ShoppingListDetailPage.xaml.cs i SettingsPage.xaml.cs odczytem wartości z lokalizatora przy użyciu resource managerów i bezpiecznego fallbacku.
  • Dodano / rozszerzono klucze w zasobach stron: HomePageResources.resx, ShoppingListDetailPageResources.resx, SettingsPageResources.resx dla wszystkich nowych komunikatów dialogowych używanych na stronach.
  • Dodano wspólne klucze przycisków (OK, Yes, No) do ButtonResources.resx i zmieniono dialogi, by korzystały z tych kluczy.
  • Wprowadzono pomocnicze metody GetHomeText, GetShoppingListText, GetSettingsText i GetButtonText w odpowiednich plikach widoków, które czytają wartości z ResourceManager z domyślnym fallbackiem.

Testing

  • Uruchomiono prosty statyczny skrypt sprawdzający brak literalnych wywołań DisplayAlert("..." w zmodyfikowanych plikach, wynik: 0 literalnych wywołań znalezionych w docelowych plikach (sukces).
  • Sprawdzono zmiany w plikach zasobów (ButtonResources.resx, HomePageResources.resx, ShoppingListDetailPageResources.resx, SettingsPageResources.resx) i widokach (HomePage.xaml.cs, ShoppingListDetailPage.xaml.cs, SettingsPage.xaml.cs) oraz zatwierdzono zmiany (commit utworzony lokalnie).
  • Próba uruchomienia dotnet build zakończyła się niepowodzeniem w tym środowisku ponieważ dotnet CLI nie jest dostępny, więc pełny build/test nie został wykonany tutaj.

Codex Task

Copy link
Copy Markdown
Owner Author

@phoneuooyd phoneuooyd left a comment

Choose a reason for hiding this comment

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

it's alright

@phoneuooyd phoneuooyd merged commit 7e7b804 into main Mar 6, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant