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

feat: change slot definition #535

Merged
merged 7 commits into from
Jul 12, 2021
Merged

feat: change slot definition #535

merged 7 commits into from
Jul 12, 2021

Conversation

farnabaz
Copy link
Collaborator

@farnabaz farnabaz commented Jul 5, 2021

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

improved syntax

::block
---
front-matter: data
format: yaml
---

Default Slot

#NamedSlot

slot content
---

highlighted syntax

Screen Shot 2021-07-05 at 8 17 29 PM

resolves nuxtlabs/vscode-mdc#1

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes (if not applicable, please state why)

@vercel
Copy link

vercel bot commented Jul 5, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/nuxtlabs/docus/xFq41xwfGmSXvFzNAPe2pz6yFZGx
✅ Preview: https://docus-git-feat-slot-syntax-nuxtlabs.vercel.app

Copy link
Contributor

atinux commented Jul 5, 2021

Nice!

We need to update the documentation + release as a breaking change (0.9.0) and let the Nuxt team update the new nuxtjs.org also

@farnabaz
Copy link
Collaborator Author

farnabaz commented Jul 5, 2021

I've updated usages in the docs and Syntax section.

@atinux atinux requested a review from Tahul July 5, 2021 23:04
@Tahul
Copy link
Contributor

Tahul commented Jul 6, 2021

Sure; this will be shipped into next breaking release; 0.9.0.

Copy link
Contributor

@Tahul Tahul left a comment

Choose a reason for hiding this comment

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

Hey @farnabaz 😄

I checked your PR, the new syntax looks great, and looks amazing when used with the Docus plugin!

I've found a problem on page "theme/components", that seem to indicate a problem there:
https://github.com/nuxtlabs/docus/blob/cf1e653299798b78c421018600ec88633be936db/src/core/runtime/utils.ts#L60

CleanShot 2021-07-12 at 00 27 55@2x

Could you take a look at that? 😄

@farnabaz
Copy link
Collaborator Author

Updated \cc @Tahul

@Tahul Tahul merged commit 734fdca into dev Jul 12, 2021
@Tahul Tahul deleted the feat/slot-syntax branch July 12, 2021 15:46
Tahul added a commit that referenced this pull request Jul 12, 2021
* 📝 (docs) update docs; add algolia docsearch in configuration

* fix(layout): fix parents order in layout inheritance

* fix: add fixed width to mobile aside (#515)

* fix: multiple regressions (#516)

* fix: margins for docs content

* fix: multiple issues

* feat: enhanced i18n (#517)

* fix(Link): support locale path & improve attrs

* fix(contentLocalePath): handle locale prefix

* 🐛 (link) fix blank reeference (props.blank)

* 🔖 (0.8.6) release 0.8.6

* 🐛 (prose-code-inline) allow line break (#519)

* fix: scrollspy & scroll to top on click for Releases (#522)

* fix: restrict inline syntax (#526)

* fix: restrict inline syntax

* fix: allow inline syntax inside span & links

* feat: auto collapse when click on another section (#527)

* init pr

* ✨ (aside) restore collapse state on section toggle

Co-authored-by: Yaël GUILLOUX <[email protected]>

* 🐛 (nuxtjs.org) small responsive/sizing fixes

* fix: prose nested lists (#529)

* fix: prose nested lists

* cleanup

* fix: multiple style issues (#523)

* fix: code block issues

* update filename shortcut

* feat: support slot to show between children (#525)

* feat: support slot to show between children

* fix: do not add `between` slot between inline nodes

* feat: inline SVG logo on production (#528)

* feat: inline svg icon on production

* fix: import

* chore: cleanup

* feat: improve logo management, add default Logo component and shortcuts

* chore: remove @nuxtjs/svg

* chore: simplify markup

Co-authored-by: Sébastien Chopin <[email protected]>

* feat: SandBox Component (#518)

* feat: SandBox Component

* chore: minor fix

* feat: init Tabs component; refactor CodeGroup

* update Sandbox UI

Co-authored-by: Sergey Bedritsky <[email protected]>

* 📦 (deps) upgrade dependencies

* 🐛 (aside) fix aside navigation collapsing

* 🐛 (aside) fix social icons spacing at the right place

* 🔖 (0.8.7) release 0.8.7

* 📦 (deps) upgrade deps (nuxt-image)

* 🐛 (sandbox) fix sandbox component

* feat: add custom scrollbar (#534)

* fix: update mobile toc overflow from scroll to auto (#532)

* 🔖 (0.8.8) release 0.8.8

* 📝 (deployment) update deployment to add npm cache

* 🔖 (0.8.9) release 0.8.9

* 🐛 (aside) hover color only in large on aside nav item

* init AppBanner (#536)

* init AppBanner

* ✨ (banner) edit defaults

* 📝 (config) update configuration with banner arg

Co-authored-by: Yaël GUILLOUX <[email protected]>

* 🔖 (0.8.10) release 0.8.10

* 🔖 (0.8.11) release 0.8.11

* 🐛 (docs) update docs links (avoid errors while building..)

* fix: Releases content overflow (#537)

* fix: Releases content overflow

* 🐛 (releases) only apply overflow-x-hidden

Co-authored-by: Yaël GUILLOUX <[email protected]>

* 🐛 (content) fix content links avoiding a longer build

* 🔖 (0.8.12) release 0.8.12

* chore: reduce UX issues (lazy loading and jump)

* fix: currentNav updated after page change

Also disabled smart prefetch for lazy components in the meantime

* chore: add isAsync too

* fix: revert back to old collapse logic (#539)

* ⏪ (collapse) revert back to old collapse logic

* 🐛 (navigation) preserve uncollapsing for first visited category

* 📌 (deps) pin vue to 2.6.14

* 🐛 (twitter) fix twitter component import

* fix: set variables on SSR from asyncData

* 🔖 (0.8.13) release 0.8.13

* 🔖 (0.8.14) release 0.8.14

* fix: resolve flash issue using render fucntion instead of template (#541)

* feat: Docus loading screen

* 🔖 (0.8.15) release 0.8.15

* fix: tokenise blogpost (#540)

* feat: change slot definition (#535)

* feat: change slot definition

* feat: front matter style data

* chore: update docs

* docs: fix docs

* chore: refactor data tokenizer

* fix: typo

* fix: invalid line detection

* feat: use remark-github (#538)

* feat(admin): admin improvements (#524)

* wip: components playground

* fix: switch to nuxt module for components preview

* refactor(admin): modulize preview

* refactor(admin): folder structure

* chore: typo

* feat(admin): embed windicss-analysis

* chore: update notes

* feat: config tab

* feat: preview navigate to editor

* fix(admin): reuse utils instance for windi analysis

* chore: update deps

* feat(admin): entry for  windi analyzer

* chore: clean up

* Apply suggestions from code review

Co-authored-by: Sébastien Chopin <[email protected]>

* 🚨 (lint) no-console fix

* chore: cleanup vite fix

Co-authored-by: Sébastien Chopin <[email protected]>
Co-authored-by: Yaël GUILLOUX <[email protected]>

Co-authored-by: Ahad Birang <[email protected]>
Co-authored-by: Sergey Bedritsky <[email protected]>
Co-authored-by: Sébastien Chopin <[email protected]>
Co-authored-by: Anthony Fu <[email protected]>
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.

feat: syntax coloring
3 participants