Skip to content

Fix #456: Localize export-to-text placeholders with German translations and E2E test#476

Closed
iclectic wants to merge 1 commit intoNeoCoderMatrix86:developmentfrom
iclectic:localize-export-placeholders-v2
Closed

Fix #456: Localize export-to-text placeholders with German translations and E2E test#476
iclectic wants to merge 1 commit intoNeoCoderMatrix86:developmentfrom
iclectic:localize-export-placeholders-v2

Conversation

@iclectic
Copy link
Copy Markdown

Description

This PR resolves issue #456 by fully localizing the export-to-text placeholders in the export dialog.

  • Updated GenerateExportDialog.razor to use plain localisation keys (Artist, Title, Album, etc.) without 'Placeholder_' prefix
  • Added German translations in GenerateExportDialog.de.resx for placeholder names
  • Added Title attributes to MudMenuItem components for tooltips showing both localised names and placeholder values
  • Extended ChangeLanguageAsync E2E test to verify German localization of export placeholder dropdown labels
  • Resolves issue Export to text placeholders are not localized #456 by making export placeholder names user-friendly and localised

…nd E2E test

- Updated GenerateExportDialog.razor to use plain localization keys (Artist, Title, Album, etc.) without 'Placeholder_' prefix
- Added German translations in GenerateExportDialog.de.resx for placeholder names
- Added Title attributes to MudMenuItem components for tooltips showing both localized names and placeholder values
- Extended ChangeLanguageAsync E2E test to verify German localization of export placeholder dropdown labels
- Resolves issue #456 by making export placeholder names user-friendly and localized
exportOptions.SelectedExportProfile.SchemeHead += placeholder.Value;
}
}">
}"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Title is not allowed in MudBlazor!

<data name="Placeholder_Artist" xml:space="preserve">
<value>Künstler</value>
</data>
<data name="Placeholder_Title" xml:space="preserve">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please do NOT use Placeholder_

<data name="Performer" xml:space="preserve">
<value>Performer</value>
</data>
<data name="Placeholder_Artist" xml:space="preserve">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please do NOT use Placeholder_

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.

2 participants