embed more video render sidar menu more times

wideweide commented 1 year ago

Bug Report

Steps to reproduce

# More video

## video 1
![video](video/1.mp4 ':include :type=video width=95% controls=controls ')

## video 2
![video](video/2.mp4 ':include :type=video width=95% controls=controls ')

## video 3
![video](video/3.mp4 ':include :type=video width=95% controls=controls ')

What is current behaviour

side bar is like this:

More video
    - video 1
    - video 2
    - video 3
    - More video
       -  video 1
        - video 2
        - video 3
    - More video
        - video 1
        - video 2
        - video 3

What is the expected behaviour

side bar is like this:

More video
    - video 1
    - video 2
    - video 3

Other relevant information

[ ] Bug does still occur when all/other plugins are disabled?
Your OS: Pop!_OS 22.04 LTS
Node.js version: v18.15.0
npm/yarn version: 9.5.0
Browser version: firefox 112.0.1
Docsify version: //cdn.jsdelivr.net/npm/docsify/lib/docsify.min.js
Docsify plugins:

Please create a reproducible sandbox

Mention the docsify version in which this bug was not present (if any)

wideweide commented 1 year ago

ps: if video/1.mp4 is not exists, just not show the video replace with real url that can be visited, can reproduce my issue

Koooooo-7 commented 1 year ago

Related to issue #1822 and pr in https://github.com/docsifyjs/docsify/pull/1824 .

wideweide commented 1 year ago

Related to issue #1822 and pr in #1824 .

I'm a newbie, so how can I use the pr to solve my issue?

Koooooo-7 commented 1 year ago

Related to issue #1822 and pr in #1824 .

I'm a newbie, so how can I use the pr to solve my issue?

Basically, I think it is not a good way to build ur own docsify dependency. and I think we need get this fix and release a path asap.

cc @docsifyjs/reviewers , PTAL at #1824.

docsifyjs / docsify