Skip to content

Conversation

@stephannv
Copy link
Contributor

Adding tailwindcss-animate plugin back to setup installer and moving logic related to javascript packages to a separated module.

@stephannv stephannv self-assigned this Nov 19, 2024
Copy link
Member

@sethhorsley sethhorsley left a comment

Choose a reason for hiding this comment

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

Looks good but we need to make sure things are still working with import maps and esbuild. I remember there was something messing up with importmaps and the animate lib.

Comment on lines +1 to +5
require_relative "javascript_utils"
module RubyUI
module Generators
class ComponentGenerator < Rails::Generators::Base
include RubyUI::Generators::JavascriptUtils
Copy link
Member

Choose a reason for hiding this comment

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

♥️

@stephannv
Copy link
Contributor Author

Looks good but we need to make sure things are still working with import maps and esbuild. I remember there was something messing up with importmaps and the animate lib.

@sethhorsley yeah, I tested on 2 apps(rails with importmap and rails with esbuild) and the animation worked fine on both apps.

@stephannv stephannv merged commit 341d585 into ruby-ui:main Nov 19, 2024
2 checks passed
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