Skip to content

POC Improve disabled editor contrast by letting Wonder Blocks own the disabled state#3761

Draft
ivyolamit wants to merge 10 commits into
mainfrom
LEMS-4131/poc-improve-editor-controls-contrast
Draft

POC Improve disabled editor contrast by letting Wonder Blocks own the disabled state#3761
ivyolamit wants to merge 10 commits into
mainfrom
LEMS-4131/poc-improve-editor-controls-contrast

Conversation

@ivyolamit

Copy link
Copy Markdown
Contributor

Summary:

Issue: LEMS-4131

Test plan:

@ivyolamit ivyolamit self-assigned this Jun 11, 2026
…dge to replace the deprecated pill component
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Size Change: +1.12 kB (+0.22%)

Total Size: 509 kB

📦 View Changed
Filename Size Change
packages/perseus-editor/dist/es/index.js 106 kB +1.06 kB (+1.01%)
packages/perseus/dist/es/index.js 200 kB +59 B (+0.03%)
ℹ️ View Unchanged
Filename Size
packages/kas/dist/es/index.js 20.6 kB
packages/keypad-context/dist/es/index.js 1 kB
packages/kmath/dist/es/index.js 6.32 kB
packages/math-input/dist/es/index.js 98.5 kB
packages/math-input/dist/es/strings.js 1.61 kB
packages/perseus-core/dist/es/index.item-splitting.js 12 kB
packages/perseus-core/dist/es/index.js 26.3 kB
packages/perseus-linter/dist/es/index.js 9.8 kB
packages/perseus-score/dist/es/index.js 10.2 kB
packages/perseus-utils/dist/es/index.js 403 B
packages/perseus/dist/es/strings.js 8.6 kB
packages/pure-markdown/dist/es/index.js 1.39 kB
packages/simple-markdown/dist/es/index.js 6.71 kB

compressed-size-action

@@ -537,6 +548,7 @@ class AnswerOption extends React.Component<
<Strut size={spacing.small_12} />

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.

Remove strut, this is deprecated WB component

);
}

const styles = StyleSheet.create({

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.

use css module

@@ -461,7 +465,11 @@ class ExpressionEditor extends React.Component<Props, State> {
<View style={{gap: spacing.xSmall_8}}>{answerOptions}</View>

<Strut size={spacing.small_12} />

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.

Do not use Strut


type ChangeFn = typeof Changeable.change;

const fieldStyles = StyleSheet.create({

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.

use css modules

editingDisabled: boolean;
};

type AnswerProps = {

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.

change to interface

cursor: "not-allowed",
},

disabledLink: {

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.

use css module

selectedStyle: {
backgroundColor: "#ddd",
},

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.

use css module

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.

1 participant