Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

Mq orientation #528

Closed
wants to merge 2 commits into from
Closed

Mq orientation #528

wants to merge 2 commits into from

Conversation

aFarkas
Copy link
Collaborator

@aFarkas aFarkas commented Jun 24, 2015

This implements (orientation: portrait|landscape) into our matchMedia polyfill.

@Wilto
Copy link
Collaborator

Wilto commented Jun 24, 2015

I’m still worried about the additional overhead of shimming media queries. It was the first thing on my mind as I was reading http://codepen.io/Tigt/blog/when-responsive-images-get-ugly the other day. There are a lot of really cool MQ things on the horizon, which—in theory—would all be available to Picturefill when using matchMedia. By shimming MQ, we’re limiting Picturefill, and committing to playing catch-up—adding to our code weight and ongoing responsibilities.

@aFarkas aFarkas closed this Jun 24, 2015
@aFarkas aFarkas deleted the mq-orientation branch June 24, 2015 16:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants