diff options
| author | Ben Harris <bjh21@bjh21.me.uk> | 2022-10-17 23:11:48 +0100 |
|---|---|---|
| committer | Ben Harris <bjh21@bjh21.me.uk> | 2022-10-17 23:17:59 +0100 |
| commit | 5c180cfa6fee3409d60d959adacc0701d16c0e26 (patch) | |
| tree | 13c7448c8d10ff3bdf5aa42ac4ac24ede8f2b69b | |
| parent | 3c3d8aff2229be103c340c9c2e235c33127556fe (diff) | |
| download | puzzles-5c180cfa6fee3409d60d959adacc0701d16c0e26.zip puzzles-5c180cfa6fee3409d60d959adacc0701d16c0e26.tar.gz puzzles-5c180cfa6fee3409d60d959adacc0701d16c0e26.tar.bz2 puzzles-5c180cfa6fee3409d60d959adacc0701d16c0e26.tar.xz | |
js: When making a hidden element visible, just remove "display: none"
This removes any assumption in the JavaScript code about precisely what
"display" setting the element should have.
This means that now the only places where the JavaScript manipulates
elements' styles are to set the width of the puzzle container and to
mark and unmark elements with "display: none". These both seem like
reasonable kinds of semantic markup that just happen to be expressed as
styles.
| -rw-r--r-- | emcclib.js | 2 | ||||
| -rw-r--r-- | emccpre.js | 2 |
2 files changed, 2 insertions, 2 deletions
@@ -173,7 +173,7 @@ mergeInto(LibraryManager.library, { } else { seed = UTF8ToString(seed); permalink_seed.href = "#" + seed; - permalink_seed.style.display = "inline"; + permalink_seed.style.display = ""; } }, @@ -522,6 +522,6 @@ function initPuzzle() { // it's probably safe to hide the 'sorry, no puzzle here' div and // show the div containing the actual puzzle. document.getElementById("apology").style.display = "none"; - document.getElementById("puzzle").style.display = "inline"; + document.getElementById("puzzle").style.display = ""; }; } |