Skip to content

Remove respond.js#48

Open
coaxial wants to merge 2 commits into
pages-themes:masterfrom
coaxial:patch-1
Open

Remove respond.js#48
coaxial wants to merge 2 commits into
pages-themes:masterfrom
coaxial:patch-1

Conversation

@coaxial

@coaxial coaxial commented Jun 11, 2023

Copy link
Copy Markdown
Contributor

CSS media queries have been supported by every major browser for a while now, there is no need for the shim anymore.

coaxial added 2 commits June 11, 2023 10:45
Every major browser now supports CSS3 media queries, no need for the shim anymore.
@coaxial

coaxial commented Jul 2, 2023

Copy link
Copy Markdown
Contributor Author

Failed because:

   *  External link https://twitter.com/mattgraham failed: 302 Number of redirects hit maximum amount

HTML-Proofer found 1 failure!

I cannot re-rerun the action to see if it passes now.

@olifre

olifre commented Mar 10, 2024

Copy link
Copy Markdown
Contributor

I cannot re-rerun the action to see if it passes now.

You may want to rebase onto current master which will re-run the action automatically, it seems the temporary problem you encountered is gone for now.

olifre added a commit to olifre/olifre.github.io that referenced this pull request Mar 10, 2024
All recent browsers work fine without it.

See also: pages-themes/midnight#48
@olifre

olifre commented Mar 10, 2024

Copy link
Copy Markdown
Contributor

@coaxial I think one could go even further and also drop jQuery and the other two browser shims in the default layout, what do you think?
jQuery does not appear to be used anywhere else in the theme.

olifre added a commit to olifre/olifre.github.io that referenced this pull request Mar 10, 2024
All recent browsers work fine without it.

See also: pages-themes/midnight#48
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