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

ISSUE-369: Make vaul use data- attr for Content direction #404

Conversation

mikita-liaukovich-epam
Copy link
Contributor

Related issue: 369

  • This PR makes vaul use common data-attribute for content direction. This allows usage of Tailwind classes to style the Drawer differently for different directions.
  • The typo vault is fixed in style.css

Copy link

vercel bot commented Sep 6, 2024

@mikita-liaukovich-epam is attempting to deploy a commit to the emil Team on Vercel.

A member of the Team first needs to authorize it.

@emilkowalski
Copy link
Owner

This is great, thank you!

Could we do that for all attributes? I can imagine people would want to style other ones as well.

@mikita-liaukovich-epam
Copy link
Contributor Author

@emilkowalski done! I updated all custom vaul- attributes with data-vaul-

@emilkowalski
Copy link
Owner

Thank you so much @mikita-liaukovich-epam

@emilkowalski emilkowalski merged commit 2ae6cf7 into emilkowalski:main Sep 7, 2024
1 check failed
kodiakhq bot referenced this pull request in technifit/tasker Sep 13, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [vaul](https://vaul.emilkowal.ski/) ([source](https://github.com/emilkowalski/vaul)) | [`0.9.2` -> `0.9.3`](https://renovatebot.com/diffs/npm/vaul/0.9.2/0.9.3) | [![age](https://developer.mend.io/api/mc/badges/age/npm/vaul/0.9.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/vaul/0.9.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/vaul/0.9.2/0.9.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/vaul/0.9.2/0.9.3?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>emilkowalski/vaul (vaul)</summary>

### [`v0.9.3`](https://github.com/emilkowalski/vaul/releases/tag/v0.9.3)

[Compare Source](https://github.com/emilkowalski/vaul/compare/v0.9.2...v0.9.3)

#### What's Changed

Most importantly the scroll top issue should now be gone, issues with controlled state/defaultOpen prop should also be fixed.

-   feat: add export on props by [@&#8203;ferdylimmm9](https://github.com/ferdylimmm9) in [https://github.com/emilkowalski/vaul/pull/405](https://github.com/emilkowalski/vaul/pull/405)
-   adjust snapPoints on window resize by [@&#8203;bviebahn](https://github.com/bviebahn) in [https://github.com/emilkowalski/vaul/pull/399](https://github.com/emilkowalski/vaul/pull/399)
-   Export DialogProps type by [@&#8203;martynaskadisa](https://github.com/martynaskadisa) in [https://github.com/emilkowalski/vaul/pull/348](https://github.com/emilkowalski/vaul/pull/348)
-   fix: scroll top by [@&#8203;emilkowalski](https://github.com/emilkowalski) in [https://github.com/emilkowalski/vaul/pull/406](https://github.com/emilkowalski/vaul/pull/406)
-   ISSUE-369: Make vaul use data- attr for Content direction by [@&#8203;mikita-liaukovich-epam](https://github.com/mikita-liaukovich-epam) in [https://github.com/emilkowalski/vaul/pull/404](https://github.com/emilkowalski/vaul/pull/404)
-   fix: controlled state by [@&#8203;emilkowalski](https://github.com/emilkowalski) in [https://github.com/emilkowalski/vaul/pull/408](https://github.com/emilkowalski/vaul/pull/408)
-   refactor: background scaling by [@&#8203;emilkowalski](https://github.com/emilkowalski) in [https://github.com/emilkowalski/vaul/pull/409](https://github.com/emilkowalski/vaul/pull/409)
-   feat: add prop to disable input repositioning by [@&#8203;emilkowalski](https://github.com/emilkowalski) in [https://github.com/emilkowalski/vaul/pull/410](https://github.com/emilkowalski/vaul/pull/410)

#### New Contributors

-   [@&#8203;ferdylimmm9](https://github.com/ferdylimmm9) made their first contribution in [https://github.com/emilkowalski/vaul/pull/405](https://github.com/emilkowalski/vaul/pull/405)
-   [@&#8203;bviebahn](https://github.com/bviebahn) made their first contribution in [https://github.com/emilkowalski/vaul/pull/399](https://github.com/emilkowalski/vaul/pull/399)
-   [@&#8203;martynaskadisa](https://github.com/martynaskadisa) made their first contribution in [https://github.com/emilkowalski/vaul/pull/348](https://github.com/emilkowalski/vaul/pull/348)
-   [@&#8203;mikita-liaukovich-epam](https://github.com/mikita-liaukovich-epam) made their first contribution in [https://github.com/emilkowalski/vaul/pull/404](https://github.com/emilkowalski/vaul/pull/404)

**Full Changelog**: emilkowalski/vaul@v0.9.2...v0.9.3

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] If you want to rebase/retry this PR, check this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/technifit/tasker).
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