Remove Inter font assets from inclusion in downstream projects

[stefanlindbohm]

This gem currently bundles the Inter font as part of its assets, which gets included in any project depending on it. As such it will also be picked up by rails assets:prepare and output into the public assets directory, whether the project uses the font or not. I'd like to propose these assets be removed from tailwindcss-rails or alternatively be included in a way that does not force assets into downstream projects.

I know Inter is often used in Tailwind examples and as a starting point for smaller projects. This makes sense. But this gem should in my mind also be the preferred way for larger Rails projects using Tailwind, where Inter is unlikely to be the font of choice. In these cases, including the font assets with no way of opting out creates unnecessary clutter in those projects & deployments.

Possible actions

1. Remove the font assets. The Inter font project has an official CDN provided by Cloudfront that could be a documented suggestion for new projects & upgrade path. This would of course be a breaking change.
2. If there's a way for Rails engines to conditionally provide assets, move to using that. I'm not aware of any such methods myself though(?).

Workaround

I'm currently hacking my way out of these assets with the following configuration. I'm open to suggestions on how to do this in a cleaner way if possible. A clean & documented way to do this might remove the need for the proposed change.

Rails.application.config.to_prepare do
  Rails.application.config.assets.precompile -= %w[inter-font.css]
  Rails.application.config.assets.paths.reject! { _1.is_a?(String) && _1.include?('gems/tailwindcss-rails') }
end

---

I'd like to hear what you (maintainers) think about this. If this is a desirable change, I'm open to working on a PR.

[robzolkos]

I've been adding the workaround snippet above to a bunch of my projects to mitigate Inter being included.

This has become more of an issue since the :all option was added to stylesheet_link_tag in Propshaft in this PR

The inter.css file will be downloaded by the browser, even if there is no Inter font usage in the app.

imo - as Inter is not a part of Tailwind per se - it should not be a part of this gem. That's probably a separate discussion. Perhaps it could be an optional inclusion.

[flavorjones]

@stefanlindbohm I think we're open to a PR to improve this.

Just to be explicit about the goals I think we're agreeing on:

• we should keep the "freshly installed" experience by shipping a good font and styling out of the box
• once developers start customizing their styling and assets, it should be easy to remove the default "inter-font.css"

The naive solution that comes to mind for me is to move the asset initialization from

tailwindcss-rails/lib/tailwindcss/engine.rb

Lines 5 to 7 in d52b49e

	initializer "tailwindcss.assets" do
	Rails.application.config.assets.precompile += %w( inter-font.css )
	end

into an app initializer that is generated during tailwindcss:install.

The only tricky part would be making the gem backwards-compatible with apps that don't have that initializer installed.

WDYT? What ideas do you have?

[dixpac]

I would remove inter font, maybe use system-ui or just leave default one which is font-family: ui-sans-serif, system-ui, sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"; (afaik)

[robzolkos]

I would remove inter font

I agree. Here is a diff of the scaffolded views with and without Inter (without uses the default Tailwind font stack). It looks great without Inter!

The only tricky part would be making the gem backwards-compatible with apps that don't have that initializer installed.

True but maybe a major version bump and an education piece on how to add it back (or a link to add fonts in general) may work?

[stefanlindbohm]

Thanks for the responses, everyone.

@flavorjones:

Just to be explicit about the goals I think we're agreeing on:

• we should keep the "freshly installed" experience by shipping a good font and styling out of the box
• once developers start customizing their styling and assets, it should be easy to remove the default "inter-font.css"

Sounds reasonable to me. I would personally add the goal that minimal complexity should be carried as part of this gem, with the assumption that the default font is just a starting point that will be customized by many/most projects. The point @robzolkos makes that Inter is not part of Tailwind per se is relevant here.

The only tricky part would be making the gem backwards-compatible with apps that don't have that initializer installed.

Right. Whether we can do breaking changes (and a major version bump) decides what paths we can take.

Steps I can see for a non-breaking change:

• Making the Rails.application.config.assets.precompile configuration optional. Not sure if separating the initializer as you suggest is possible without breaking apps? The only way I can think of off the top of my head is to introduce configuration options for this gem that allows an opt-out of the Inter font.
• Avoiding inclusion of font & stylesheet files in assets paths when opted out. This is where I'm not well-versed enough with Rails engines to know exactly. Maybe by using a non-standard assets path that is not included by Rails automatically and explicitly include the non-standard path when not opted out?

A breaking change could be done by:

• Removing fonts & stylesheet from load paths entirely.
• Adding necessary files as part of the setup templates, either using the current files as they are (just putting these files in the app's assets directories) or by simplifying to a single CSS file using the official CDN as mentioned earlier. (This would mirror the current inclusion of some Tailwind plugins in the templated Tailwind config, which I appreciated having control over.)
• Documenting an upgrade path that installs these assets into existing/upgrading apps.

I would vote for doing the breaking change as that would simplify things as well as making these defaults explicit in newly setup apps (I suspect that many apps might already be using other fonts & are unknowingly deploying/loading unused assets). What do you think about this?

[dixpac]

I think could make it work without the "breaking change", but I agree with @robzolkos and @stefanlindbohm . Some day this "baggage" will have to be dropped sooner or latter.

[robzolkos]

Circling back to this - I think dropping the inclusion of this font (which is not part of Tailwind) - could be part of an update when Tailwind4 is released sometime this year. It's a major version bump so would be a good time.

[searls]

I noticed my user agent downloading Inter last week and figured this was something that came along with Tailwind itself (or maybe the typography plugin), so FWIW I was a little surprised to stumble on this issue/discussion and find it was part of the gem. (I share that just as a "path of least surprise" note -- it makes it feel like the font's inclusion by default might be beyond one's expected scope of the gem)

If tailwind@4 is releasing soon, that might be a good time to drop it, but even in the meantime it'd be great to be able to exclude it without resorting to manually munging the precompile paths in an initializer (for fear someone forgets about it and years later it causes an issue)

I wrote a slightly more conservative version of the initializer @stefanlindbohm contributed above as a workaround -- didn't want to risk it being forgotten on the next major bump:

# config/initializers/remove_inter_font_from_tailwind_rails.rb
if Gem::Version.new(Tailwindcss::VERSION).segments.first < 3
  Rails.application.config.to_prepare do
    Rails.application.config.assets.precompile -= ["inter-font.css"]
    Rails.application.config.assets.paths.reject! { |p| p.is_a?(String) && p.include?("gems/tailwindcss-rails") }
  end
else
  raise "Reassess the need (and safety) of this hack once Tailwindcss-rails is updated to Tailwindcss 3: https://github.com/rails/tailwindcss-rails/discussions/353"
end

[taylorthurlow]

Unfortunately the font wasn't removed from the gem as of the release of version 3. This discussion thread is really the only reference to this problem, as this discussion was originally an opened issue, so I suppose it shouldn't be a surprise that there hasn't been any movement or discussion on this. Perhaps the appropriate thing would be to open a PR.

[flavorjones]

As I said in #353 (comment), we are open to a PR to address this.

[taylorthurlow]

Yep - sorry if I sounded like I was trying to assign blame anywhere, I wouldn't expect anyone to do anything about this unless there was a ready-to-go PR in place.

[flavorjones]

Planning to drop Inter as part of the v4 upgrade/update, very soon.

[searls]

Read the upgrade announcement yesterday and grateful for your work maintaining this gem—it's probably going to be a lot of work!