Added canvas theme for tableComponent in core component - #106
Conversation
There was a problem hiding this comment.
Review by claude :
Here's my assessment:
No breaking changes. The PR adds a brand-new file (_table.scss) and new SVG assets — nothing existing is modified except appending one @import line to theme.scss. All SCSS variables used ($dark-gray, $silver, $panel-bg-color, $wizard-bg-color, $very-light-gray, $focus-ring-color, $white) are defined in _variables.scss.
The only thing worth noting (non-breaking, cosmetic):
The three sort SVGs have fill="#ff13dc" (hot pink) on an opacity-0 rect — looks like a debug artifact from the SVG editor, but since opacity="0" it's invisible at runtime.
.cell-wrapper { width: 80% } at line 214 of _table.scss is a generic class name that could collide with other styles, but it's not breaking anything today.
LGTM — safe to merge.
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: