Change classes and id's to hyphen-delimited strings

Mclilzee / zap-simulator

Zap Matrix Simulator

https://mclilzee.github.io/zap-simulator/

Other

4 stars 16 forks source link

Change classes and id's to hyphen-delimited strings #47

Closed BlizZard-bot closed 2 years ago

BlizZard-bot commented 2 years ago

Right now, the classes and ids in the CSS use camelCase, they should be changed to hyphen-delimited strings

What needs to be done

Change the class and id names in the HTML to hyphen-delimited strings (class-name instead of className)
Update style.css to use the new ids and classes
Update index.js to use the new ids and classes

DigitalKnight0 commented 2 years ago

We might consider holding this one as well till the refactor since it requires changing them on quite a few places. Splitting the Js up would surely make it alot easier to navigate stuff around

BlizZard-bot commented 2 years ago

We might consider holding this one as well till the refactor since it requires changing them on quite a few places. Splitting the Js up would surely make it alot easier to navigate stuff around

Yeah, that's what I thought as well

BlizZard-bot commented 2 years ago

I'd love to work on this if its up for grabs