Skip to content

Conversation

@matyasf
Copy link
Collaborator

@matyasf matyasf commented Jan 27, 2026

To test:

  • check the new examples
  • check the changed pages for console errors

@matyasf matyasf self-assigned this Jan 27, 2026
</InstUISettingsProvider>
```

### Overriding theme for a specific component in a subtree
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

these are handled as array keys, so if there is a duplicate like here React will emit an error

Comment on lines +386 to +391
legacy: {
radiusSmall: '2rem', // View border radius
},
strokeWidth: {
sm: '1rem' // View border width
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These will likely change, I just wanted to make sure that the current state works

@github-actions
Copy link

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://instructure.design/pr-preview/pr-2382/

Built to branch gh-pages at 2026-01-27 15:50 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Contributor

@joyenjoyer joyenjoyer left a comment

Choose a reason for hiding this comment

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

nice work

<View focusWithin>
if the <code>focusWithin</code> prop is <code>true</code>, the View will display the focus ring if any of its descendants receives focus
<div tabindex="0" role="button" style={{outline: 'none'}}>Tab here to see the focus outline</div>
<div tabIndex="0" role="button" style={{outline: 'none'}}>Tab here to see the focus outline</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

I shouldn't be capitalized in tabindex


### Theme overrides for focus rings, border radii, shadows, spacing

Certain visuals as now styled via a central object called `SharedTokens`. You can override this too, for example:
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a small grammatical thing: as OF now. At least I think so

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.

4 participants