Skip to content

Remove lodash dependency and minor cleanup#2

Open
thenickberry wants to merge 7 commits into
bpgoldsb:mainfrom
thenickberry:main
Open

Remove lodash dependency and minor cleanup#2
thenickberry wants to merge 7 commits into
bpgoldsb:mainfrom
thenickberry:main

Conversation

@thenickberry

@thenickberry thenickberry commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Summary

remove unnecessary dependency:

  • lodash can be replaced with pure Javascript

cleanup in style.css:

  • px was missing from height/width
  • removed button and .cardBank

general:

  • unused comments and references were removed
  • added alt tag to for accessibility

gary additions:

  • Kasparov
  • Gygax
  • Larson
  • Winthorpe

Testing

% eslint tierlist.js && echo $?
0
% stylelint style.css && echo $?     
0
% html-validate index.html && echo $?
0

@thenickberry

Copy link
Copy Markdown
Contributor Author

I did not include the config files for eslint or stylelint, or the instructions for them and html-validate since it added files to the repo and initially felt like overkill. I can add to this PR if you like.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant