Skip to content

Release 4.8.0 break things, e.g. https://jhildenbiddle.github.io/docsify-tabs tab doesn't work any more. #663

Closed

Description

@dryruner
No description provided.

Activity

  1. dryruner commented on Oct 31, 2018

    @dryruner
    Author

    Also, if I'd like to download and import docsify.min.js@4.7.1, where could I find it?

  2. dryruner commented on Oct 31, 2018

    @dryruner
    Author

    Also, if I'd like to download and import docsify.min.js@4.7.1, where could I find it?

    Reply myself in case this is useful to others:
    Use
    <script src="https://unpkg.com/docsify@4.7.1/lib/docsify.min.js"></script>
    Instead of
    <script src="https://unpkg.com/docsify/lib/docsify.min.js"></script>

  3. QingWei-Li commented on Oct 31, 2018

    @QingWei-Li
    Member

    👆Cool! You first found a workaround.
    You can specify the version number, for example

    <script src=//unpkg.com/docsify@4.7.0></script>

    I will check it.

  4. dryruner commented on Oct 31, 2018

    @dryruner
    Author

    Thanks!

  5. QingWei-Li commented on Oct 31, 2018

    @QingWei-Li
    Member

    fixed in 4.8.1

  6. QingWei-Li commented on Nov 1, 2018

    @QingWei-Li
    Member

    @jhildenbiddle

    This DOM has changed and caused docsify-tab to not work, can you fix it?
    Now all html code inline elements will be in the p tag.(This is a bug before marked and has now been fixed.) So button will also be in the p tag.

    https://github.com/markedjs/marked/blob/f1ddca7febad16eb7861af0a7135989df0eb8e80/lib/marked.js#L59-L64

    I was compatible with this issue in 4.8.1, but I found that it caused other bugs #677, so I rolled back it.

  7. jhildenbiddle commented on Nov 1, 2018

    @jhildenbiddle
    Member

    Fixed in docsify-tabs 1.0.6.

    Thanks for the heads up, @QingWei-Li!

  8. QingWei-Li commented on Nov 1, 2018

    @QingWei-Li
    Member

    You are right. I reread the marked source code, it does not wrap the block-level elements.

  9. jhildenbiddle commented on Nov 1, 2018

    @jhildenbiddle
    Member

    Hehe...

    That's why I deleted my comment about wrapping all HTML in a <p>. Turns out <button> tags are wrapped, but not all HTML. 😄

  10. added a commit that references this issue on Jan 20, 2020
  11. added a commit that references this issue on Jun 9, 2024
    d35059d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugconfirmed as a bug

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions