Skip to content

docs(playground): minor clean up in the playground's tabs - #48

Merged
danielroe merged 3 commits into
nuxt-modules:mainfrom
azrikahar:minor-fixes
Aug 31, 2022
Merged

docs(playground): minor clean up in the playground's tabs#48
danielroe merged 3 commits into
nuxt-modules:mainfrom
azrikahar:minor-fixes

Conversation

@azrikahar

Copy link
Copy Markdown
Contributor

Minor clean up & consistency changes to the playground's tabs.

Tab1

  • remove add ionicon import as it seems to not be used in the tab
  • remove the class from <ion-toolbar> just as a clean up and I believe it's not currently in use

Tab2

  • remove ionicon imports and use them directly as they're auto imported
  • import UserPhoto type for showActionSheet
  • <ion-col>'s :key was using the photo object directly, updated it to use photo.filepath instead

Tab4

@vercel

vercelBot commented Aug 31, 2022

Copy link
Copy Markdown

Someone is attempting to deploy a commit to a Personal Account owned by @danielroe on Vercel.

@danielroe first needs to authorize it.

@vercel

vercelBot commented Aug 31, 2022

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewUpdated
nuxt-ionic✅ Ready (Inspect)Visit PreviewAug 31, 2022 at 2:45PM (UTC)

@danielroedanielroe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thank you very much!

@danielroedanielroe changed the title refactor(playground): Minor clean up in the playground's tabsdocs(playground): minor clean up in the playground's tabsAug 31, 2022
@danielroe
danielroe merged commit 3657e86 into nuxt-modules:mainAug 31, 2022
@azrikahar
azrikahar deleted the minor-fixes branch August 31, 2022 14:59
@github-actionsgithub-actionsBot mentioned this pull request Jun 26, 2026
Sign up for freeto 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

@azrikahar@danielroe