Skip to content

Conversation

@OrbisK
Copy link
Member

@OrbisK OrbisK commented Apr 13, 2025

This PR adds a simple mobile menu to prevent unusable settings + add a button to directly report issues at vueuse with reproduction url

@OrbisK OrbisK requested a review from 43081j April 13, 2025 20:11
@netlify
Copy link

netlify bot commented Apr 13, 2025

Deploy Preview for playground-vueuse ready!

Name Link
🔨 Latest commit f0238ce
🔍 Latest deploy log https://app.netlify.com/projects/playground-vueuse/deploys/687a5f9395873c0008599751
😎 Deploy Preview https://deploy-preview-11--playground-vueuse.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 project configuration.

@OrbisK OrbisK requested a review from ilyaliao May 17, 2025 10:21
@ilyaliao
Copy link
Member

It looks good to me, would it be better to place the menu on the right?

CleanShot 2025-05-25 at 09 24 20@2x

@43081j
Copy link

43081j commented Jul 18, 2025

it should be in the top left or top right. so in this case we should put it in the top right i think 👍

@OrbisK
Copy link
Member Author

OrbisK commented Jul 18, 2025

I think we should wait until nuxt ui pro is free (september). I have a license but I wanted to keep it OSS fiendly.

Nuxt UI Pros UMain and UHeader includes this behavior.

@43081j
Copy link

43081j commented Jul 18, 2025

can we not just reorder the elements? 👀

are you saying that nuxt defines the order so we can't change it?

@OrbisK
Copy link
Member Author

OrbisK commented Jul 18, 2025

can we not just reorder the elements? 👀

are you saying that nuxt defines the order so we can't change it?

No, you're right. I am currently upgrading the playground to Nuxt 4 to resolve the assets patch. That's why I did some refactoring. We should merge this for the v3 versions, but there are some refactorings with v4. That's why I considered not porting it to v4, because v4 might still take some time 😅 Sorry for the confusion

@OrbisK OrbisK merged commit 65c357b into main Jul 18, 2025
4 checks passed
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.

4 participants