jb-tlm / color-charter

https://color-charter.vercel.app
MIT License
0 stars 0 forks source link

Edit comments and variable names to reflect our name changes #24

Closed jb-tlm closed 1 year ago

jb-tlm commented 1 year ago

In toolbar.js, the first section is called "History Controls." This is the old terminology for the Add button. This wording was changed because there is no actual history and this was confusing for users, refreshing will wipe everything and we don't want students thinking that their palettes will be saved. Starting at the very top, change the History section name and variable names to more accurately reflect our "add" & "remove" names instead of "history +" and "history -" this should be carefully done throughout the entire toolbar.js file (there are some sessionHistory variables that shouldn't be changed), but not on main.js. All of the "history" instances on main.js are related to the session and shouldn't be changed. Please note that changing any variable name will require you to search all the files ( ctrl + shift + F ) for that variable and make sure that all instances are changed. This can be done pretty easily and quickly with "find and replace" by hitting the arrow to the left of the search bar, which opens up a text box for "replace"You can search for something in the top box, and replace any instances with whatever you put in the bottom box. However, please make sure you go through them one by one and make sure the change is appropriate instead of just hitting "replace all"