Skip to content

Update en.json#272

Merged
MathMan05 merged 7 commits intoMathMan05:mainfrom
TheGeekno72:main
Mar 3, 2026
Merged

Update en.json#272
MathMan05 merged 7 commits intoMathMan05:mainfrom
TheGeekno72:main

Conversation

@TheGeekno72
Copy link
Contributor

Description

Improving on text cues and feature descriptions - fifth pass

browsed as much pages and checked as much UI elements as I could all at once, went over the whole file again, cross compared with existing translation to make sure they fully made sense, some clearer text again, grammatical simplifications/fixes & some flavor edits too

in order to avoid Yet Another PR, hopefully the last major revision on this file, fingers crossed! 🤞😅👍

Improving on text cues and feature descriptions - fifth pass

in order to avoid Yet Another PR, I browsed as much pages and checked as much UI elements as I could, went over the whole file again, cross compared with existing translation to make sure they made sense, fixed remaining text that could cause ambiguous translation works, some clearer text again & grammatical simplifications/fixes

hopefully the last major revision, fingers crossed!
removing empty line
"colors": {
"name": "Color count:",
"one": "One",
"one": "1",
Copy link
Owner

Choose a reason for hiding this comment

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

I think using the actual words is better here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

about that, I -personally- don't like "one/two/three", I think a numerical selection would be better
BUT
I was considering submitting "monochrome/bicolor/tricolor", what do you think?

could submit that question to someone else maybe?

Copy link
Owner

Choose a reason for hiding this comment

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

I don't think having the actual numbers is fully ideal, though monochrome does not feel like the right word either. Though I'd be open for a discution somewhere else

Copy link
Contributor Author

Choose a reason for hiding this comment

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

replaced monochrome with unicolor

Choose a reason for hiding this comment

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

monochrome deffo feels more appropriate to me personally

Copy link
Owner

Choose a reason for hiding this comment

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

I think unicolor is worse, though I don't know what would be good

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IMHO, if an amount has to be described, then numbers (1, 2, 3) should be used
packing whole words in a drop down selector sounds unjustified
(especially when you have to account for the translation behind, why translate whole word numbers "just" for that?)

current : one, two, three
in PR : 1, 2, 3
alternative : unicolor, bicolor, tricolor

Copy link
Owner

Choose a reason for hiding this comment

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

yk what I've changed my mind, monochrome is fine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yk what I've changed my mind, monochrome is fine

done :)

following first reviews of current PR
following second reviews of current PR
small patch for consistency in permission descriptions
attempt to resolve last reviews
following today's review from Mathium
@MathMan05 MathMan05 merged commit 9a2a694 into MathMan05:main Mar 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants