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: refactor composables #4391

Open
wants to merge 36 commits into
base: develop
Choose a base branch
from

Conversation

m0ksem
Copy link
Collaborator

@m0ksem m0ksem commented Oct 11, 2024

This PR is intended to make it easier to use composables, making them using the same format and code style.

Right now, most of the composables made spontaneously. When composable is moved to std it means it is in good state.

I split composables into:

  • internal - used to work with vue or vuestic services.
  • ssr - used to optimize framework when working with ssr. Ideally, make eslint rule to prevent using window, document in favor of useWindow, useDocument.
  • browser - used to work with browser api and DOM elements.

Copy link

netlify bot commented Oct 11, 2024

👷 Deploy request for vuestic-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 115b199

Copy link

netlify bot commented Oct 11, 2024

Deploy Preview for vuestic-storybook ready!

Name Link
🔨 Latest commit 115b199
🔍 Latest deploy log https://app.netlify.com/sites/vuestic-storybook/deploys/673599414c9c6e0008f7a334
😎 Deploy Preview https://deploy-preview-4391--vuestic-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

1 participant