-
Notifications
You must be signed in to change notification settings - Fork 577
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
Added PL translation for Shortcut settings pane #1861
base: main
Are you sure you want to change the base?
Added PL translation for Shortcut settings pane #1861
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Of course, I didn't actually check the Polish.
I have one minor suggestion.
src/main/resources/resources/logisim/strings/gui/gui.properties
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MarcinOrlowski for this localization work!
hotkeySimTickHalf = Manual tick half cycle | ||
hotkeySimTickFull = Manual tick full cycle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't it be "Manually tick..."?
@davidhutchens: Could you maybe comment on this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that you point it out, it does seem slightly off. I would actually delete "Manual" and just write "Tick half cycle" and "Tick full cycle". Manual(ly) provides no additional information.
But note that "Manual Tick Half Cycle" is the name of the menu item. So either way, we should be consistent. I might just leave the PR as is. The labels in the panel all begin with uppercase and follow with lowercase. The menu items start each word with uppercase. I don't think the case difference matters, but I wouldn't use different words.
hotkeyDirNorth = ↑ North | ||
hotkeyDirSouth = ↓ South | ||
hotkeyDirEast = → East | ||
hotkeyDirWest = ← West |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure if adding the arrows is beneficial.
↑ North
← West ↓ South → East
is IMO not better than
North
West South East
If we decide that we like the arrows, we should consider moving →
to the right of East
(East →
)
No description provided.