From 5b55a2f894f688629e2d5ba3821847aaccb55010 Mon Sep 17 00:00:00 2001
From: Clare Ming <cming@wikimedia.org>
Date: Thu, 4 Aug 2022 15:20:58 -0600
Subject: [PATCH] Minor fixes to TOC hide/show buttons

Bug: T307901
Change-Id: I37c6b3b4c5c20e914423c311795dbccce2a7b3fd
---
 includes/templates/TableOfContents.mustache      |  8 ++------
 .../components/TableOfContentsCollapsed.less     |  1 +
 .../__snapshots__/tableOfContents.test.js.snap   | 16 ++++------------
 3 files changed, 7 insertions(+), 18 deletions(-)

diff --git a/includes/templates/TableOfContents.mustache b/includes/templates/TableOfContents.mustache
index 05185701..689fedcd 100644
--- a/includes/templates/TableOfContents.mustache
+++ b/includes/templates/TableOfContents.mustache
@@ -2,12 +2,8 @@
 	<div id="sidebar-toc-label" class="sidebar-toc-header">
 		<p class="sidebar-toc-title">
 		{{ msg-vector-toc-heading }}
-		<button class="vector-toc-uncollapse-button">
-			{{ msg-vector-toc-toggle-position-sidebar }}
-		</button>
-		<button class="vector-toc-collapse-button">
-			{{ msg-vector-toc-toggle-position-title }}
-		</button>
+		<button class="vector-toc-uncollapse-button">{{msg-vector-toc-toggle-position-sidebar}}</button>
+		<button class="vector-toc-collapse-button">{{msg-vector-toc-toggle-position-title}}</button>
 		</p>
 	</div>
 	<ul class="sidebar-toc-contents">
diff --git a/resources/skins.vector.styles/components/TableOfContentsCollapsed.less b/resources/skins.vector.styles/components/TableOfContentsCollapsed.less
index 80298e54..e6b65108 100644
--- a/resources/skins.vector.styles/components/TableOfContentsCollapsed.less
+++ b/resources/skins.vector.styles/components/TableOfContentsCollapsed.less
@@ -68,6 +68,7 @@
 	padding: 0;
 	background-color: transparent;
 	color: @color-primary;
+	cursor: pointer;
 
 	&:hover {
 		color: @color-primary--hover;
diff --git a/tests/jest/__snapshots__/tableOfContents.test.js.snap b/tests/jest/__snapshots__/tableOfContents.test.js.snap
index dd88d8e2..ce090838 100644
--- a/tests/jest/__snapshots__/tableOfContents.test.js.snap
+++ b/tests/jest/__snapshots__/tableOfContents.test.js.snap
@@ -5,12 +5,8 @@ exports[`Table of contents renders when \`vector-is-collapse-sections-enabled\`
 	<div id=\\"sidebar-toc-label\\" class=\\"sidebar-toc-header\\">
 		<p class=\\"sidebar-toc-title\\">
 		Contents
-		<button class=\\"vector-toc-uncollapse-button\\">
-			
-		</button>
-		<button class=\\"vector-toc-collapse-button\\">
-			
-		</button>
+		<button class=\\"vector-toc-uncollapse-button\\"></button>
+		<button class=\\"vector-toc-collapse-button\\"></button>
 		</p>
 	</div>
 	<ul class=\\"sidebar-toc-contents\\">
@@ -72,12 +68,8 @@ exports[`Table of contents renders when \`vector-is-collapse-sections-enabled\`
 	<div id=\\"sidebar-toc-label\\" class=\\"sidebar-toc-header\\">
 		<p class=\\"sidebar-toc-title\\">
 		Contents
-		<button class=\\"vector-toc-uncollapse-button\\">
-			
-		</button>
-		<button class=\\"vector-toc-collapse-button\\">
-			
-		</button>
+		<button class=\\"vector-toc-uncollapse-button\\"></button>
+		<button class=\\"vector-toc-collapse-button\\"></button>
 		</p>
 	</div>
 	<ul class=\\"sidebar-toc-contents\\">