936622 - remove media queries for text and popup events

Status: VERIFIED FIXED

(Webmaker Graveyard :: Popcorn Maker, defect)

Description

[Scott [:thecount] Downe]

I decided to split bug 913729 up.

This bug changes the underlaying way we resize text, but doesn't actually remove media queries.

With this patch nothing should change from the user perspective, it just means we're no longer locked to media query sizes.

Comment 1

[Scott [:thecount] Downe]

Attachment: https://github.com/mozilla/popcorn.webmaker.org/pull/306

Comment 2

[Matthew Schranz [:mjschranz]]

Comment on attachment 829475 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/306

There's some initial comments, but the initial calc seems wrong? It could also be because of a typo in your code.

Right after the event is placed:
https://www.dropbox.com/s/ymc5pab69s5s4jp/Screenshot%202013-11-08%2015.27.53.png

After I open my console and close it again:
https://www.dropbox.com/s/45xl5uxio5xkvz1/Screenshot%202013-11-08%2015.28.46.png

Comment 3

[Scott [:thecount] Downe]

Comment on attachment 829475 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/306

Yeah, that was totally caused by the typo.

We should be good now.

Comment 4

[Matthew Schranz [:mjschranz]]

Comment on attachment 829475 [details] [review]
https://github.com/mozilla/popcorn.webmaker.org/pull/306

R+ with one comment nit.

Comment 5

[Scott [:thecount] Downe]

Staged: https://github.com/mozilla/popcorn.webmaker.org/commit/8b6128727f7285c471f668ca3a9a27ff944a2a45

So far all this does is make text and popup event's position work while in full screen and using the _ embed link.

Now though, there is nothing stopping us from removing the set video sizes in the editor and embed shell. Now we can do whatever we want.

Needs verification.