SchwarzIT / onyx

🚀 A design system and Vue.js component library created by Schwarz IT
https://onyx.schwarz
Apache License 2.0
40 stars 4 forks source link

refactor(OnyxTable): rename property `grid` to `withVerticalBorders` #1297

Closed larsrickert closed 1 week ago

larsrickert commented 1 week ago

Relates to #1266

Reviewer question: Should we also rename the striped property or is that fine?

We got feedback that the property name grid might be mixed up with the "real" grid so we rename it here

Checklist

changeset-bot[bot] commented 1 week ago

🦋 Changeset detected

Latest commit: f93dbb226e0831b947201cc1c58ef8cd26036781

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages | Name | Type | | ------------------------- | ----- | | sit-onyx | Major | | alpha-test-app | Patch | | playground | Patch | | @sit-onyx/chartjs-plugin | Major | | @sit-onyx/nuxt | Major | | @sit-onyx/storybook-utils | Major | | @sit-onyx/vitepress-theme | Major |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR