Skip to content

fix(app): clean layout#1822

Merged
atinux merged 13 commits intomainfrom
pr/1820
Mar 23, 2025
Merged

fix(app): clean layout#1822
atinux merged 13 commits intomainfrom
pr/1820

Conversation

@benjamincanac
Copy link
Member

@benjamincanac benjamincanac commented Mar 21, 2025 •
edited
Loading

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

  1. remove useTrackEvent handling in app.vue
  2. remove UMain in docs layout as its already defined in app.vue
  3. clean error.vue to behave like app.vue

@nuxthub-admin
Copy link
Contributor

nuxthub-admin bot commented Mar 21, 2025 •
edited
Loading

✅ Deployed nuxt-com

Deployed nuxt-com 997c37d to preview

🔗 pr-1820.nuxt-com-uoq.pages.dev
📌 884b6192.nuxt-com-uoq.pages.dev
📱
View QR Code QR code linking to deployment URL.

📋 View deployment logs

@benjamincanac benjamincanac changed the title fix(app): remove client only around content search fix(app): enable search with cmd+k Mar 22, 2025
@benjamincanac benjamincanac changed the title fix(app): enable search with cmd+k fix(app): clean layout Mar 23, 2025
@benjamincanac benjamincanac requested a review from atinux March 23, 2025 14:03
@benjamincanac benjamincanac marked this pull request as ready for review March 23, 2025 14:05
@atinux atinux merged commit 9b62cab into main Mar 23, 2025
15 of 16 checks passed
@atinux atinux deleted the pr/1820 branch March 23, 2025 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@atinux atinux atinux approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@benjamincanac @atinux