Skip to content

Conversation

@mxmou
Copy link
Member

@mxmou mxmou commented Nov 29, 2023

Resolves #6663

Changes

Makes blocks2image use black text and icons when high contrast mode is enabled.

Reason for changes

To make the text readable.

Tests

Tested on Edge and Firefox.

@mxmou mxmou added the scope: addon Related to one or multiple addons label Nov 29, 2023
@mxmou mxmou self-assigned this Nov 29, 2023
@mxmou mxmou force-pushed the blocks2image-contrast branch from e457cf6 to 5d24adc Compare November 29, 2023 16:06
Copy link
Member

@DNin01 DNin01 left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@DNin01 DNin01 added the status: needs review PR needs 1 more approval label Nov 29, 2023
Copy link
Member

@WorldLanguages WorldLanguages left a comment

Choose a reason for hiding this comment

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

Fixes the bug on Chromium, original color theme continues to work properly

@WorldLanguages WorldLanguages removed the status: needs review PR needs 1 more approval label Nov 29, 2023
@mxmou mxmou merged commit f595f3b into ScratchAddons:master Nov 30, 2023
@mxmou mxmou deleted the blocks2image-contrast branch November 30, 2023 07:35
Samq64 pushed a commit to Jazza-231/ScratchAddons that referenced this pull request Feb 26, 2024
@WorldLanguages
Copy link
Member

WorldLanguages commented Jul 25, 2024

Fetching the same image multiple times can become slow. Thanks @Jazza-231 and @Samq64 for noticing this.

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

Labels

scope: addon Related to one or multiple addons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Text in exported blocks don't follow color mode setting

4 participants