Refactoring of the code for the navigation
Today the navigation is full custom.
For example :
bloc1.addEventListener('click', (ev) => {
document.querySelector('#subscription-form form [name="screen2"]').classList.remove("visible");
screen3.classList.remove("visible");
screen4.classList.remove("visible");
screen4b.classList.remove("visible");
screen1.classList.add("visible");
bloc1.classList.add("active");
bloc2.classList.remove("active");
bloc3.classList.remove("active");
bloc4.classList.remove("active");
});
It's a shame because we don't use our native router. I have opened a issue in sib-router to be able to disable a route of some condition aren't fulfilled => framework/sib-router#76
Other suggestion : I think it's a bad practice to set a "visible" class while it exists a native html attribute to do the job : "hidden".