TOC should hide on empty `.dynamic-content` (not empty `ul`)

groton-school / knowledgebase

Knowledgebase website

https://kb.groton.org

GNU General Public License v3.0

0 stars 0 forks source link

TOC should hide on empty `.dynamic-content` (not empty `ul`) #28

Open github-actions[bot] opened 1 month ago

github-actions[bot] commented 1 month ago

https://github.com/groton-school/knowledgebase/blob/757961371913d04e9cf50d11977469ef225df443/apps/ui/src/UI/TOC/styles.scss#L1


// TODO should be .dynamic-content not ul
#toc.card:has(ul:empty) {
  display: none;
}

battis commented 1 month ago

The TOC was originally built to slot into Overdrive's DOM. Serious tweaks to make it more streamlined will break that. Holding off on those tweaks. For now.