fix: sidebar active class and expand don't work as expect when use "s…

…pace" in markdown filename (#1454) * fix: decode href in sidebar (#1032) * Create sidebar.test.js Co-authored-by: 沈唁 <[email protected]>
docsifyjs · Dec 20, 2020 · dcf5a64 · dcf5a64 · vercel · Dec 20, 2020
1 parent 76c5e68
commit dcf5a64
Show file tree

Hide file tree

Showing 2 changed files with 71 additions and 1 deletion.
diff --git a/src/core/event/sidebar.js b/src/core/event/sidebar.js
@@ -83,7 +83,7 @@ export function getAndActive(router, el, isParent, autoTitle) {
   links
     .sort((a, b) => b.href.length - a.href.length)
     .forEach(a => {
-      const href = a.getAttribute('href');
+      const href = decodeURI(a.getAttribute('href'));
       const node = isParent ? a.parentNode : a;
 
       a.title = a.title || a.innerText;

diff --git a/test/e2e/sidebar.test.js b/test/e2e/sidebar.test.js
@@ -0,0 +1,70 @@
+const docsifyInit = require('../helpers/docsify-init');
+
+// Suite
+// -----------------------------------------------------------------------------
+describe('Sidebar Tests', function() {
+  // Tests
+  // ---------------------------------------------------------------------------
+  test('Active Test', async () => {
+    const docsifyInitConfig = {
+      markdown: {
+        sidebar: `
+          - [Test Space](test%20space)
+          - [Test _](test_foo)
+          - [Test -](test-foo)
+          - [Test .](test.foo)
+          - [Test >](test>foo)
+          - [Test](test)
+        `,
+      },
+      routes: {
+        '/test space.md': `
+          # Test Space
+        `,
+        '/test_foo.md': `
+          # Test _
+        `,
+        '/test-foo.md': `
+          # Test -
+        `,
+        '/test.foo.md': `
+          # Test .
+        `,
+        '/test>foo.md': `
+          # Test >
+        `,
+        '/test.md': `
+          # Test page
+        `,
+      },
+    };
+
+    await docsifyInit(docsifyInitConfig);
+    await page.click('a[href="#/test%20space"]');
+    await expect(page).toEqualText(
+      '.sidebar-nav li[class=active]',
+      'Test Space'
+    );
+    expect(page.url()).toMatch(/\/test%20space$/);
+
+    await page.click('a[href="#/test_foo"]');
+    await expect(page).toEqualText('.sidebar-nav li[class=active]', 'Test _');
+    expect(page.url()).toMatch(/\/test_foo$/);
+
+    await page.click('a[href="#/test-foo"]');
+    await expect(page).toEqualText('.sidebar-nav li[class=active]', 'Test -');
+    expect(page.url()).toMatch(/\/test-foo$/);
+
+    await page.click('a[href="#/test.foo"]');
+    await expect(page).toEqualText('.sidebar-nav li[class=active]', 'Test .');
+    expect(page.url()).toMatch(/\/test.foo$/);
+
+    await page.click('a[href="#/test>foo"]');
+    await expect(page).toEqualText('.sidebar-nav li[class=active]', 'Test >');
+    expect(page.url()).toMatch(/\/test%3Efoo$/);
+
+    await page.click('a[href="#/test"]');
+    await expect(page).toEqualText('.sidebar-nav li[class=active]', 'Test');
+    expect(page.url()).toMatch(/\/test$/);
+  });
+});