Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Update interface references #6068

Merged
merged 4 commits into from
Dec 16, 2022

Conversation

macandcheese
Copy link
Contributor

Removes individually specified Appearance interfaces and instead uses Extract<> to source values from a single defined reference.

@macandcheese macandcheese requested a review from a team as a code owner December 16, 2022 00:24
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Dec 16, 2022
@@ -1,7 +1,5 @@
import type Color from "color";

export type ColorAppearance = "default" | "minimal" | "solid";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I couldn't find any references to this "default" value.

Copy link
Contributor

Choose a reason for hiding this comment

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

default is deprecated here. I think its safe to remove it.

Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

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

💯 Awesome. Super nice that these will all be using the same interface for consistency.

@macandcheese macandcheese merged commit f388ad5 into master Dec 16, 2022
@macandcheese macandcheese deleted the macandcheese/cleanup-appearance-references branch December 16, 2022 01:03
@github-actions github-actions bot added this to the 2023 January Priorities milestone Dec 16, 2022
benelan added a commit that referenced this pull request Dec 16, 2022
* master: (36 commits)
  1.0.0-next.696
  feat(tip,tip-manager): add built-in translations (#6074)
  1.0.0-next.695
  fix(icon, graphic, loader): Set aria-hidden on internal svg elements (#6069)
  1.0.0-next.694
  refactor(modal)!: Renamed CSS variable (#6078)
  1.0.0-next.693
  refactor(color-picker-hex-input)!: removed `intl*` properties (#6063)
  1.0.0-next.692
  feat(shell): Add slots for Modal and Alert (#5983)
  1.0.0-next.691
  refactor(input-date-picker)!: Removed deprecated start, startAsDate, end, endAsDate properties (#6076)
  1.0.0-next.690
  fix(combobox): Fix error when typing a custom value (#6071)
  1.0.0-next.689
  fix(rating): 5312 improve user interface (#5948)
  chore: Clean outdated properties (#6073)
  docs: update component READMEs (#6072)
  chore: Update interface references (#6068)
  chore(action): Clean up action appearance references (#6067)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants