Display warning if JavaScript is disabled, fixes #15 #17
Conversation
|
J'ai essayé d'utiliser bootstrap au maximum, tout en essayant d'intégrer au mieux le message dans la page déjà existant. |
| <div id="noscript" class="d-flex align-items-center flex-column justify-content-center"> | ||
| <p> |
There was a problem hiding this comment.
| <div id="noscript" class="d-flex align-items-center flex-column justify-content-center"> | |
| <p> | |
| <p id="noscript" class="d-flex align-items-center flex-column justify-content-center"> |
There was a problem hiding this comment.
Je n'ai pas rebuild pour vérifier mais tu as bien checké que l'affichage reste toujours le même via ce changement (propriétés par défaut du div et du p qui ne sont pas les mêmes ?) ?
| <a href="https://www.enable-javascript.com/fr/">Cliquez ici</a> pour accéder aux instructions | ||
| pour activer JavaScript dans votre navigateur. | ||
| </p> | ||
| </div> |
| @@ -55,6 +55,17 @@ h2 { | |||
| font-size: large; | |||
| } | |||
|
|
|||
| #noscript { | |||
| margin: 250px 0 250px 0; | |||
There was a problem hiding this comment.
| margin: 250px 0 250px 0; | |
| margin: 250rem 0 250rem 0; |
| p { | ||
| text-align: center; | ||
| font-weight: bold; | ||
| font-size: 1.3em; |
There was a problem hiding this comment.
| font-size: 1.3em; | |
| font-size: 1.3rem; |
There was a problem hiding this comment.
Quel avantage les rem ont-ils par rapport aux em ?
There was a problem hiding this comment.
https://caniuse.com/#feat=mdn-api_css_em
Le support des rem semble meilleur, non ?
| document.addEventListener("DOMContentLoaded", function() { | ||
| document.querySelector("#wrapper").classList.remove("d-none"); | ||
| }); |
There was a problem hiding this comment.
Cela me semble overkill d'ajouter du JS, j'aurai mis le texte en rouge.
There was a problem hiding this comment.
Ca évite toute confusion au moins.
There was a problem hiding this comment.
Tu aurais mis quelle partie du texte en rouge ?
|
Je veux bien continuer d'éditer cette PR mais je ne pense pas qu'ils la mergent plus tard. |
This PR fixes #15, allowing users to get instructions on how to enable JavaScript for their browser in the case it is disabled.
This will enable more people to use the generator.