TurboWarp
cursor: translate cursor menu
Resolves: https://github.com/TurboWarp/extensions/issues/1987
I changed some naming or cursors, lmk if some others should be changed as well...
If you are going to do this preferably add an index for each item like native scratch, so people who dont know english (or the right value cause you changed the naming) can still use the block effectively.
that or we need to make a new block and use a field instead
done
The formatting bot didn't find any formatting issues. It currently only checks the extensions folder. The author or a maintainer can run terminal command 'npm run format' manually to format all files.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
GarboMuffin in #1987:
if the names don't match the internal CSS names anymore, a lot of them could have much better names
The current names are meant to parody those used in the CSS "cursor" attribute. These names are not localized to any language except English, and so there's not really a reason to translate them unless we plan to change the names to make them more descriptive. I don't see anything wrong with doing so.
before doing that, as their is obviously conflicting opinions, can you consult with the other people...
before doing that, as their is obviously conflicting opinions, can you consult with the other people...
im fine with no numbers
before doing that, as their is obviously conflicting opinions, can you consult with the other people...
im fine with no numbers
that isnt the point. there are other reviewers on this team besides just you and garbomuffin... and this is already a precedence set by scratchfoundation for their translated accept reporters menus.. which this so keenly breaks on purpose now; with no consultation to anyone else before hand in which the PR is involved.
before doing that, as their is obviously conflicting opinions, can you consult with the other people...
im fine with no numbers
that isnt the point. there are other reviewers on this team besides just you and garbomuffin... and this is already a precedence set by scratchfoundation for their translated accept reporters menus.. which this so keenly breaks on purpose now; with no consultation to anyone else before hand in which the PR is involved.
No other reviewers voiced a concern for numbers. Additionally, no other extension in the TW gallery follows that number format.
Sure get the other's opinions, but there doesnt seem to be a point
before doing that, as their is obviously conflicting opinions, can you consult with the other people...
im fine with no numbers
that isnt the point. there are other reviewers on this team besides just you and garbomuffin... and this is already a precedence set by scratchfoundation for their translated accept reporters menus.. which this so keenly breaks on purpose now; with no consultation to anyone else before hand in which the PR is involved.
No other reviewers voiced a concern for numbers. Additionally, no other extension in the TW gallery follows that number format.
Sure get the other's opinions, but there doesnt seem to be a point
Additionally, no other extension in the TW gallery follows that number format.
"this is already a precedence set by scratchfoundation" "breaks on purpose now"
there is a point, you are choosing to ignore it. https://github.com/TurboWarp/extensions/pull/2076#discussion_r2048179943 https://github.com/TurboWarp/extensions/pull/2076#pullrequestreview-2755039521
further talk on this subject was done in the discord: https://discord.com/channels/837024174865776680/1147654296926687242/1321630894204649502 https://discord.com/channels/837024174865776680/1147654296926687242/1321634779040448512 https://discord.com/channels/837024174865776680/1147654296926687242/1321637065439580202
We already have many extensions that have translatable menus without a number fallback or equivalent: Animated Text (when enabling features incompatible with Scratch Lab), Files, Pointer lock, Runtime options, Text, Clipping and blending, Iframe, Pen+ V7, Skins, Sensing+, Key simulation, Mobile keyboard, I stopped looking but there are lots more
Merging this without numbers also does not preclude the future possibility of adding numbers
Feel free to revert my commit and make the numbers actually work though. I'll approve that
So was this going to be merged or were we going to address the indexed items translation issue that doesn't seem to me like an actual issue
