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
fix: i18n of gtk msgbox buttons #19904
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.
lgtm with comment about potentially DRYing this out!
Also, 4-2-x
is security-only at this juncture so i don't think we'd want to backport this there :)
rename from `gtk_util.[cc,h]` to `util_gtk.[cc,h]` so that it gets picked up by the `extra_source_filters` rule in `BUILD.gn`.
It really shows that I cannot build locally atm... :P
Release Notes Persisted
|
I was unable to backport this PR to "5-0-x" cleanly; |
I was unable to backport this PR to "6-0-x" cleanly; |
I have automatically backported this PR to "7-0-x", please check out #19982 |
@ckerr could you please backport these to 6/5 when you have a moment? |
Manually backport #19904. See that PR for details.
A maintainer has manually backported this PR to "6-0-x", please check out #20007 |
Manually backport #19904. See that PR for details.
A maintainer has manually backported this PR to "5-0-x", please check out #20010 |
* fix: i18n of gtk msgbox buttons Manually backport #19904. See that PR for details. * fix: make linter happy * fix: make linter happy
* fix: i18n of gtk msgbox buttons Manually backport #19904. See that PR for details. * fix: make linter happy * fix: make linter happy
Description of Change
Does for GTK messageboxes what #19756 (12df0e8) did for GTK file dialogs: fixes the i18n of the stock buttons in a way that doesn't break the buttons' keyboard shortcuts.
CC @codebytere
Checklist
npm test
passesRelease Notes
Notes: Fixed i18n of GTK messageboxes' stock buttons.