From 5c107bfe4dcba7cd955f2087957a46d1323b3b05 Mon Sep 17 00:00:00 2001
From: Kumaravel Palanisamy <kumar.okm1995@gmail.com>
Date: Sat, 16 May 2020 20:35:05 +0530
Subject: [PATCH 1/2] Tooltip added for sidemenu subitems

---
 src/core/render/tpl.js        |  7 ++++++-
 src/themes/basic/_layout.styl | 34 ++++++++++++++++++++++++++++++++++
 src/themes/vue.styl           |  4 ++++
 3 files changed, 44 insertions(+), 1 deletion(-)

diff --git a/src/core/render/tpl.js b/src/core/render/tpl.js
index 85efa436c..168a4c90c 100644
--- a/src/core/render/tpl.js
+++ b/src/core/render/tpl.js
@@ -93,7 +93,12 @@ export function tree(toc, tpl = '<ul class="app-sub-sidebar">{inner}</ul>') {
 
   let innerHTML = '';
   toc.forEach(node => {
-    innerHTML += `<li><a class="section-link" href="${node.slug}">${node.title}</a></li>`;
+    innerHTML += `<li>
+        <div class="tooltip">
+          <span class="tooltiptext">${node.title}</span>
+          <a class="section-link" href="${node.slug}">${node.title}</a>
+        </div>
+      </li>`;
     if (node.children) {
       innerHTML += tree(node.children, tpl);
     }
diff --git a/src/themes/basic/_layout.styl b/src/themes/basic/_layout.styl
index 3d6d5c33b..8fbd60377 100644
--- a/src/themes/basic/_layout.styl
+++ b/src/themes/basic/_layout.styl
@@ -12,6 +12,40 @@ body:not(.ready)
   [data-cloak], .app-nav, > nav
     display none
 
+.tooltip {
+	position: relative;
+}
+
+.tooltip .tooltiptext {
+	visibility: hidden;
+	opacity: 0;
+	transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
+	-webkit-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
+	-o-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
+	-moz-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
+	visibility: hidden;
+	background-color: black;
+	color: #fff;
+	text-align: center;
+	border-radius: 6px;
+	padding: 5px 5px;
+	position: absolute;
+	z-index: 21;
+	bottom: 100%;
+	left: 50%;
+	margin-left: -105px;
+}
+
+.tooltip:hover .tooltiptext {
+	visibility: visible;
+	opacity: 1;
+}
+
+.tooltip .tooltiptext:hover {
+	visibility: hidden;
+}
+
+
 div#app
   font-size 30px
   font-weight lighter
diff --git a/src/themes/vue.styl b/src/themes/vue.styl
index 54f161de4..877771677 100644
--- a/src/themes/vue.styl
+++ b/src/themes/vue.styl
@@ -11,6 +11,10 @@ $sidebar-width = 300px
 body
   background-color $color-bg
 
+.tooltip .tooltiptext {
+  margin-left: -120px;
+}
+
 /* sidebar */
 .sidebar
   background-color $color-bg

From f8a17b9d8a857d84a767dbe00e92c21616601f58 Mon Sep 17 00:00:00 2001
From: Kumaravel Palanisamy <kumar.okm1995@gmail.com>
Date: Sat, 13 Jun 2020 12:37:50 +0530
Subject: [PATCH 2/2] Review changes - added title for side menu sub item

---
 src/core/render/tpl.js        |  7 +------
 src/themes/basic/_layout.styl | 34 ----------------------------------
 src/themes/vue.styl           |  4 ----
 3 files changed, 1 insertion(+), 44 deletions(-)

diff --git a/src/core/render/tpl.js b/src/core/render/tpl.js
index 168a4c90c..4101bb691 100644
--- a/src/core/render/tpl.js
+++ b/src/core/render/tpl.js
@@ -93,12 +93,7 @@ export function tree(toc, tpl = '<ul class="app-sub-sidebar">{inner}</ul>') {
 
   let innerHTML = '';
   toc.forEach(node => {
-    innerHTML += `<li>
-        <div class="tooltip">
-          <span class="tooltiptext">${node.title}</span>
-          <a class="section-link" href="${node.slug}">${node.title}</a>
-        </div>
-      </li>`;
+    innerHTML += `<li><a class="section-link" href="${node.slug}" title="${node.title}">${node.title}</a></li>`;
     if (node.children) {
       innerHTML += tree(node.children, tpl);
     }
diff --git a/src/themes/basic/_layout.styl b/src/themes/basic/_layout.styl
index 8fbd60377..3d6d5c33b 100644
--- a/src/themes/basic/_layout.styl
+++ b/src/themes/basic/_layout.styl
@@ -12,40 +12,6 @@ body:not(.ready)
   [data-cloak], .app-nav, > nav
     display none
 
-.tooltip {
-	position: relative;
-}
-
-.tooltip .tooltiptext {
-	visibility: hidden;
-	opacity: 0;
-	transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
-	-webkit-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
-	-o-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
-	-moz-transition: visibility 0.3s linear 1s, opacity 0.3s linear 1s;
-	visibility: hidden;
-	background-color: black;
-	color: #fff;
-	text-align: center;
-	border-radius: 6px;
-	padding: 5px 5px;
-	position: absolute;
-	z-index: 21;
-	bottom: 100%;
-	left: 50%;
-	margin-left: -105px;
-}
-
-.tooltip:hover .tooltiptext {
-	visibility: visible;
-	opacity: 1;
-}
-
-.tooltip .tooltiptext:hover {
-	visibility: hidden;
-}
-
-
 div#app
   font-size 30px
   font-weight lighter
diff --git a/src/themes/vue.styl b/src/themes/vue.styl
index 877771677..54f161de4 100644
--- a/src/themes/vue.styl
+++ b/src/themes/vue.styl
@@ -11,10 +11,6 @@ $sidebar-width = 300px
 body
   background-color $color-bg
 
-.tooltip .tooltiptext {
-  margin-left: -120px;
-}
-
 /* sidebar */
 .sidebar
   background-color $color-bg