Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates heroicon / replaces it with rails-heroicon #52

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

robacarp
Copy link
Contributor

@robacarp robacarp commented Jan 11, 2024

The gem "heroicon" hasn't been updated in >18mo, and the heroicon pack seems to have changed a lot since then.

Another gem "rails-heroicon" seems to share a lot of the code but is actually bundled with a railtie and has carried updates as recently as 2 weeks ago. The interface is identical from the views, but it comes with all the names for icons which match the current heroicon website.

I've removed the heroicon helper and initializer. Neither are required by rails-heroicon.

Fixes this warning message spamming the console:

image

The gem "heroicon" hasn't been updated in 18mo, and the heroicon pack has dramatically changed since then.

Another gem "rails-heroicon" seems to share a lot of the code but is actually bundled with a railtie and has carried updates as recently as 2 weeks ago. The interface is identical from the views, but it comes with all the names for icons which match the current heroicon website.

I've removed the heroicon helper and initializer. Neither are required by rails-heroicon.
@@ -11,7 +11,7 @@
<p>Settings</p>
</button>
<a href="/logout" class='flex space-x-2 p-2 hover:bg-black/80 mx-2 rounded text-white text-sm items-center'>
<%= heroicon "logout" , options: { class: 'h-5 w-5 text-gray-300' } %>
<%= heroicon "arrow-right-start-on-rectangle" , options: { class: 'h-5 w-5 text-gray-300' } %>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There now no icon called logout and there was some discussion about it all. Apparently people disagree with what looks like a logout arrow and what doesn't 😂

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i intentionally added the old version since it had icons that looked similar to old chatgpt and yeah that icon is different. :D

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea, I was noting that as well. But the odd thing is that it does look like the community is backing the v2 a little stronger than v1 (keeping libraries more updated, etc). The creator of v2 also talks about it as if it's his new baby — it's the new "modern" icon library. So I did a pass in my branch to try and move everything to the v2 icons and I got it looking pretty nice so we can just stick with it now.

@krschacht krschacht merged commit 273b00c into main Jan 11, 2024
2 checks passed
@krschacht krschacht deleted the updates-heroicon branch January 11, 2024 22:24
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.

3 participants