From 0eefd63a1310d674411f688f08a08bbe55712fa0 Mon Sep 17 00:00:00 2001 From: Owen Nelson Date: Thu, 16 Jul 2020 19:27:18 -0700 Subject: [PATCH] Change "show hidden lines" icon to "eye" instead of "expand." In a recent discussion in the amethyst docs discord channel, it was suggested that using an "eye" icon might make the show hidden lines feature of mdbook's code sample rendering more discoverable. I myself overlooked the arrows that are in use now. Fixes #663 at least in part. --- src/theme/book.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/theme/book.js b/src/theme/book.js index 049505fc6d..5e386369f7 100644 --- a/src/theme/book.js +++ b/src/theme/book.js @@ -175,23 +175,23 @@ function playground_text(playground) { var buttons = document.createElement('div'); buttons.className = 'buttons'; - buttons.innerHTML = ""; + buttons.innerHTML = ""; // add expand button var pre_block = block.parentNode; pre_block.insertBefore(buttons, pre_block.firstChild); pre_block.querySelector('.buttons').addEventListener('click', function (e) { - if (e.target.classList.contains('fa-expand')) { - e.target.classList.remove('fa-expand'); - e.target.classList.add('fa-compress'); + if (e.target.classList.contains('fa-eye')) { + e.target.classList.remove('fa-eye'); + e.target.classList.add('fa-eye-slash'); e.target.title = 'Hide lines'; e.target.setAttribute('aria-label', e.target.title); block.classList.remove('hide-boring'); - } else if (e.target.classList.contains('fa-compress')) { - e.target.classList.remove('fa-compress'); - e.target.classList.add('fa-expand'); + } else if (e.target.classList.contains('fa-eye-slash')) { + e.target.classList.remove('fa-eye-slash'); + e.target.classList.add('fa-eye'); e.target.title = 'Show hidden lines'; e.target.setAttribute('aria-label', e.target.title);