Skip to content

AO3-7550 i18n-ing skins wizard font, skins wizard font size and skins wizard vertical gap help files - #5960

Open
ASalbenblatt wants to merge 9 commits into
otwcode:masterfrom
ASalbenblatt:AO3-7550_i18n-skins-wizard-font,-skins-wizard-font-size-and-skins-wizard-vertical-gap-help-files
Open

ASalbenblatt wants to merge 9 commits into
otwcode:masterfrom
ASalbenblatt:AO3-7550_i18n-skins-wizard-font,-skins-wizard-font-size-and-skins-wizard-vertical-gap-help-files

Conversation

@ASalbenblatt

Copy link
Copy Markdown
Contributor

Pull Request Checklist

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

@Bilka2
Bilka2 self-requested a review October 18, 2026 07:12

@Bilka2 Bilka2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for working on this! I have only some nitpicks

Comment on lines +20 to +21
skins_wizard_font_size
skins_wizard_font

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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)

@ASalbenblatt ASalbenblatt Oct 24, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Even though that would make this list no longer match how the files themselves are automatically ordered in the folder? (due to _ coming before .)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 😂

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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.

Comment thread app/views/help/skins_wizard_font.html.erb Outdated
Comment thread app/views/help/skins_wizard_vertical_gap.html.erb Outdated
Comment thread config/locales/views/en.yml Outdated
Comment thread config/locales/views/en.yml Outdated
Comment thread app/views/help/skins_wizard_font_size.html.erb Outdated
Co-authored-by: Bilka <Bilka2@users.noreply.github.com>
ASalbenblatt and others added 5 commits October 24, 2026 16:19
@ASalbenblatt
ASalbenblatt marked this pull request as ready for review October 27, 2026 18:32

@Bilka2 Bilka2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants