Skip to content

Conversation

@distalx
Copy link

@distalx distalx commented Apr 7, 2022

Here's a list of things that could be improved.

  • Migrating from jQuery to vanilla JS.
  • Upgrage fourseven:scss to latest.
  • Replace FontAwesome in favour of Bootstrap Icons!?
    - [ ] Remove jQuery from tests.
  • Bump package version.

The latest version of FontAwesome requires the creation of an account with them just to use the free version of it. I've been using the Bootstrap Icons for a while and I believe they can be a good replacement.

I haven't removed jQuery from the tests. Just so you can test the changes against them. For me, all tests are passing.

@rglover
Copy link
Contributor

rglover commented Apr 7, 2022

@distalx thanks for putting this together. Would you be interested in being a collaborator on the repo and taking over maintenance? Will have to figure out how to allow you to do pushes to Atmosphere...

@distalx
Copy link
Author

distalx commented Apr 7, 2022

@rglover, Thanks for the quick response.
I can help you with maintenance tasks.
Should I go ahead and make the proposed changes?

@distalx
Copy link
Author

distalx commented Apr 7, 2022

For me all tests are green.

Screenshot 2022-04-08 at 4 41 53 AM

Keeping a jQuery for test-only dependency as of now.

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.

2 participants