Over in #1471382: Add IE-conditional classes to html.tpl.php there is currently a discussion about adding IE-specific layouts to D8 core themes using IE-specific classes to the HTML element. When respond.js is thrown into that kind of mix, it causes mayhem since the IE-specific layouts apply in addition to the newly-enabled media query-specific layouts.

I can only see this being an issue if you need respond.js for a contrib theme, but are also using a core theme in parts of your site. Or if you are dealing with a site that has legacy themes and new themes with a mix of IE-conditonal-classes/respond.js solutions.

It would be nice if this module had a setting to only add the respond.js to specific themes.

Comments

rupl’s picture

Is there an example of how to do this in another module? It seems to me that the onus is on the site admin to know when any particular module is useful or harmful for the site they are building.

If only themes could have dependencies.. that would be grand.

johnalbin’s picture

It's an edge case, for sure.

I can only see this being an issue if you need respond.js for a contrib theme, but are also using a core theme in parts of your site. Or if you are dealing with a site that has legacy themes and new themes with a mix of IE-conditonal-classes/respond.js solutions.

And its an edge case that I'm not experiencing. Feel free to ignore this issue until someone with this particular itch wants to work on it.

As for theme dependencies, there's a core issue for that. :-) #474684: Allow themes to declare dependencies on modules

astonvictor’s picture

Issue summary: View changes
Status: Active » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.