Skip to content

[Minor] Rename colors variable to theme in useTheme() #678

@sundaramramaswamy

Description

@sundaramramaswamy

This came-up in code review of #654. We want to follow explicit naming for better readability. All instances of

  const {colors} = useTheme();
  // later usage
  colors.border

should be renamed to

  const {theme} = useTheme();
  // later usage
  theme.border

There are too many instances to take it up as part of #654 so creating a separate bug.
image

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions