search

BetaMasaheft / BetMas

Exist-db application of the Beta Masaheft project
6 stars 2 forks source link

Dynamic show/hide text #6

Closed smaugustine closed 3 years ago

smaugustine commented 3 years ago

This simply toggles the text in buttons that show/hide elements between "show" and "hide" instead of the current static text "hide/show".

smaugustine commented 3 years ago

A suggestion I might have would be to also store the state in local storage so that a user doesn't need to click the show/hide button each time the page loads.

PietroLiuzzo commented 3 years ago

A suggestion I might have would be to also store the state in local storage so that a user doesn't need to click the show/hide button each time the page loads.

if you want to do that, I will test it

smaugustine commented 3 years ago

Looking over w3.js, I wonder that all of the .click functions should all be inside of a $(document).ready(function(){ ... });. With jQuery, there can be problems with event handlers outside of this.