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

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

Merged
merged 1 commit into from
Jun 29, 2021
Merged

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

merged 1 commit into from
Jun 29, 2021

Conversation

Tahul
Copy link
Contributor

@Tahul Tahul commented Jun 28, 2021

image

Page:
https://admiring-heyrovsky-27a7cf.netlify.app/docs/configuration-glossary/configuration-css

Reported from the @atinux Slack message.

I've tried fixing this via browser console, removing that:
https://github.com/nuxtlabs/docus/blob/faef8fb688322ec293d6e4c8fe820d626444b30c/src/defaultTheme/components/atoms/prose/ProseCodeInline.vue#L9

Seem not to break neither the line-wrapping occurrences and the non line-wrapping ones.

@bdrtsky; could you review this to be sure it does not have side effects ?

@vercel
Copy link

vercel bot commented Jun 28, 2021

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

docus – ./

🔍 Inspect: https://vercel.com/nuxtlabs/docus/9gnc52PnbFRy1hRTYo6JoVZ3MnDh
✅ Preview: https://docus-git-fix-code-inline-nuxtlabs.vercel.app

nuxtjs-with-docus – ./

🔍 Inspect: https://vercel.com/nuxtlabs/nuxtjs-with-docus/BgS8D3ERVCfqRxoGsDdn482Zm9ey
✅ Preview: https://nuxtjs-with-docus-git-fix-code-inline-nuxtlabs.vercel.app

@bdrtsky
Copy link
Contributor

bdrtsky commented Jun 29, 2021

I actually added whitespace-nowrap just few days ago, since noticed wrapped inline code looks pretty ugly. But I guess this current issue is more important, so this pr should be ok.

@Tahul Tahul merged commit 6aa60cc into dev Jun 29, 2021
@Tahul Tahul deleted the fix/code-inline branch June 29, 2021 16:01
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.

2 participants