Repository navigation
AO3-7550 i18n-ing skins wizard font, skins wizard font size and skins wizard vertical gap help files - #5960
Conversation
… wizard vertical gap help files
Bilka2
left a comment
There was a problem hiding this comment.
Thanks for working on this! I have only some nitpicks
| skins_wizard_font_size | ||
| skins_wizard_font |
There was a problem hiding this comment.
Following the ancient tag wrangling wisdom of "nothing before something", these two should switch places to be alphabetical please (and in routes.rb as well)
There was a problem hiding this comment.
Even though that would make this list no longer match how the files themselves are automatically ordered in the folder? (due to _ coming before .)
There was a problem hiding this comment.
That's a good point and I had to take a bit to think about it. I think in this case both reasonings are perfectly sensible so I'll leave it up to you what you go for. I'll just force future PRs to match yours so we're consistent 😂
There was a problem hiding this comment.
I think leave it like this, because I feel like the only reason you'd really go looking through this list is if you were also modifying the files in some way, so I feel like they should match.
Co-authored-by: Bilka <Bilka2@users.noreply.github.com>
…-and-skins-wizard-vertical-gap-help-files' of https://github.com/ASalbenblatt/otwarchive into AO3-7550_i18n-skins-wizard-font,-skins-wizard-font-size-and-skins-wizard-vertical-gap-help-files
Co-authored-by: Bilka <Bilka2@users.noreply.github.com>
Pull Request Checklist
as the first thing in your pull request title (e.g.
AO3-1234 Fix thing)until they are reviewed and merged before creating new pull requests.
Issue
https://otwarchive.atlassian.net/browse/AO3-7550
Purpose
Internationalizes the skins wizard font, skins wizard font size and skins wizard vertical gap help files
Credit
FlyingFalcon they/them