Skip to content

Split off debugger category string - #9120

Open
Samq64 wants to merge 1 commit into
ScratchAddons:masterfrom
Samq64:blocks-category-general
Open

Split off debugger category string#9120
Samq64 wants to merge 1 commit into
ScratchAddons:masterfrom
Samq64:blocks-category-general

Conversation

@Samq64

@Samq64 Samq64 commented Jul 24, 2026

Copy link
Copy Markdown
Member

In preparation for #8382

Splits the debugger category label into its own _general string so the addon API (and userscripts) no longer rely on metadata strings. As far as I know that was the only case where it did.

@Samq64
Samq64 requested a review from WorldLanguages as a code owner July 24, 2026 19:04
@Samq64 Samq64 added scope: addon api Related to the addon.* JS APIs or other ways for addons to provide features scope: l10n All about localization/internationalization size: extra small Can be reviewed in a few minutes labels Jul 24, 2026
"_general/blocks/green-flag": "flag",
"_general/blocks/clockwise": "clockwise",
"_general/blocks/anticlockwise": "anti-clockwise",
"_general/blocks/sa-category": "Debugger",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you think a developer comment would be helpful here (to explain to translators how this string is different from the addon name)?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sure, maybe renaming the key to debugger-category would also help?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Or debugger-addon-category.

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

Labels

scope: addon api Related to the addon.* JS APIs or other ways for addons to provide features scope: l10n All about localization/internationalization size: extra small Can be reviewed in a few minutes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants