qwik-ui icon indicating copy to clipboard operation
qwik-ui copied to clipboard

FIX, DOCS

Open serbyxp opened this issue 10 months ago • 2 comments

What is it?

  • [ ] Feature / enhancement
  • [x] Bug
  • [x] Docs / tests
  • [ ] Other

Why is it needed?

Old data-state=“open” data-state=“closed” for accordion animations in global.css, and for styled Modal in-lined tailwind classes.

Updated documentation to reflect changes in Modal.css

Checklist:

  • [x] My code follows the developer guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have ran pnpm change and documented my changes
  • [x] I have add necessary docs (if needed)
  • [?] Added new tests to cover the fix / functionality

serbyxp avatar Feb 24 '25 00:02 serbyxp

🦋 Changeset detected

Latest commit: 1d732e475ea8bf55bae48d5ac8e12ae2dd087c0f

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

This PR includes changesets to release 2 packages
Name Type
qwik-ui Patch
@qwik-ui/styled Patch

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

changeset-bot[bot] avatar Feb 24 '25 00:02 changeset-bot[bot]

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
:white_check_mark: @maiieul
:x: @seanperez-xyz
You can retrigger this bot by commenting recheck in this Pull Request

github-actions[bot] avatar Feb 24 '25 00:02 github-actions[bot]

LGTM 👌 Thanks @serbyxp! I just need you to approve the CLA and I can merge the PR 🙌

maiieul avatar May 21 '25 20:05 maiieul

I have read the CLA Document and I hereby sign the CLA

serbyxp avatar May 23 '25 21:05 serbyxp

LGTM 👌 Thanks @serbyxp! I just need you to approve the CLA and I can merge the PR 🙌

I have read the CLA Document and I hereby sign the CLA

serbyxp avatar May 23 '25 21:05 serbyxp

Open in StackBlitz

npm i https://pkg.pr.new/qwikifiers/qwik-ui@1050
npm i https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/headless@1050
npm i https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/styled@1050
npm i https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/utils@1050

commit: 36b3cfe

pkg-pr-new[bot] avatar May 24 '25 18:05 pkg-pr-new[bot]