Skip to content

Code refactoring#30

Open
Rchrdlss3 wants to merge 1 commit intomainfrom
refactoring
Open

Code refactoring#30
Rchrdlss3 wants to merge 1 commit intomainfrom
refactoring

Conversation

@Rchrdlss3
Copy link
Contributor

@Rchrdlss3 Rchrdlss3 commented Oct 24, 2024

Doing some code refactoring

  • We should not have magic numbers.
  • We want to always identify is that if there is multiple occurrences in code, it is best to make it reusable instead of constantly retyping the same thing
  • Some of my naming conventions could be better, but please review while I am working on this to give better suggestions.
  • Always remember to try and keep things maintainable as possible, especially as application continues to scale.
  • Created constant var for bottom margins
9 results in 8 files - Open in editor JS AutoScrollHomeSection js archiv  1 Lemon MILK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant