Skip to content

Conversation

NathanBaulch
Copy link
Contributor

Found 22 static strings that weren't localizable. I took a guess at the key names, feel free to rename.

@NathanBaulch NathanBaulch changed the title localization: move 22 strings into resources localization: move strings into resources Jul 7, 2025
@xavierdono
Copy link

App.Text add "Text." before.

App.Text("Text.InteractiveRebase.Pick") it's just App.Text("InteractiveRebase.Pick"), no?

@NathanBaulch
Copy link
Contributor Author

Good point @xavierdono, fixed!

flyout.Placement = PlacementMode.BottomEdgeAlignedLeft;
flyout.VerticalOffset = -4;

CreateActionMenuItem(flyout, Brushes.Green, "Pick", "Use this commit", "P", item, Models.InteractiveRebaseAction.Pick);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Action name like Pick, Squash etc. should not be translated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, in that case can we use the enum name?

@love-linger love-linger merged commit 1d31f5f into sourcegit-scm:develop Jul 11, 2025
13 checks passed
@love-linger love-linger self-assigned this Jul 11, 2025
@love-linger love-linger added the localization Localization/Translation label Jul 11, 2025
@NathanBaulch NathanBaulch deleted the localization branch July 11, 2025 04:11
love-linger added a commit that referenced this pull request Jul 11, 2025
- Do not translate interactive rebase action description.
- For slider label in `BLEND` diff mode, it can not use `Text.Diff.Binary.New` or `Text.Diff.Binary.Old`. Because those keys are labels for size.

Signed-off-by: leo <[email protected]>
love-linger added a commit that referenced this pull request Jul 11, 2025
- Do not translate interactive rebase action description.
- For slider label in `BLEND` diff mode, it can not use `Text.Diff.Binary.New` or `Text.Diff.Binary.Old`. Because those keys are labels for size.

Signed-off-by: leo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localization Localization/Translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants