Skip to content

Add icon overrides; remove MessageButton; rename TextButton ghost to subtle - #122

Merged
pszerzo-amazon merged 1 commit into
mainfrom
icon-overrides
Oct 26, 2026
Merged

pszerzo-amazon merged 1 commit into
mainfrom
icon-overrides

Conversation

@pszerzo-amazon

@pszerzo-amazon pszerzo-amazon commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Icon overrides: new icons option (also settable via update()) replaces any built-in icon by its Icons key. Components read icons through a new useIcons hook; the merged map is memoized, so identities stay stable across re-renders.
  • createIcon(children, svgProps?): builds a custom icon with a 24x24 viewBox and currentColor fill, and applies size, className and aria-hidden. Exported with Icon, IconProps, IconName and IconOverrides.
  • TextButton: subtle is the new default variant; ghost still works as an alias.
  • MessageButton: removed (component, exports, design-system specimen).
  • README documents icons.

Notes

  • accent20 is intentionally unchanged; its usages are still to be decided.
  • docs/ is not regenerated (npm run update-readme).
  • Playground mocks still import icons directly, so overrides don't affect them.

@pszerzo-amazon pszerzo-amazon changed the title Icon overrides Add icon overrides; remove MessageButton; rename TextButton ghost to subtle Oct 26, 2026
@pszerzo-amazon
pszerzo-amazon force-pushed the icon-overrides branch 2 times, most recently from c8cd719 to f753ed9 Compare October 26, 2026 17:49
sderacy
sderacy previously approved these changes Oct 26, 2026
…subtle

- Add `icons` option (also via `update()`) to replace any built-in icon by its
  `Icons` key; components read icons through a new `useIcons` hook.
- Add `createIcon(children, svgProps?)`: 24x24 viewBox, `currentColor` fill,
  handles `size`/`className`/`aria-hidden`. Export it and the icon types.
- TextButton: `subtle` is the new default variant; `ghost` remains an alias.
- Remove MessageButton (component, exports, specimen).
- Document icons in the README.

Test
@pszerzo-amazon
pszerzo-amazon merged commit fbc41ce into main Oct 26, 2026
4 checks passed
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.

2 participants