Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "mejs-shim" class to all shims to prevent improper resizing #789

Merged
merged 3 commits into from Mar 11, 2013
Merged

Add "mejs-shim" class to all shims to prevent improper resizing #789

merged 3 commits into from Mar 11, 2013

Conversation

JeffreyATW
Copy link
Contributor

We have a use case in which foreign iframes (overlay ads) are inserted into MediaElementPlayer's DOM structure. They need to remain a constant size, but there is some code that resizes all <object>s, <iframe>s, and <embed>s to 100% at times.

I've changed this code so every shim is created with a mejs-shim class, and they are selected as such, instead of selecting specific kinds of elements.

johndyer added a commit that referenced this pull request Mar 11, 2013
Add "mejs-shim" class to all shims to prevent improper resizing
@johndyer johndyer merged commit 67b7cf9 into mediaelement:master Mar 11, 2013
@JeffreyATW JeffreyATW deleted the shim_class branch March 12, 2013 05:55
phuongdh pushed a commit to avalonmediasystem/mediaelement that referenced this pull request Oct 18, 2013
Add "mejs-shim" class to all shims to prevent improper resizing
rexblack pushed a commit to rexblack/mediaelement that referenced this pull request May 22, 2014
Add "mejs-shim" class to all shims to prevent improper resizing
rexblack pushed a commit to rexblack/mediaelement that referenced this pull request May 22, 2014
Add "mejs-shim" class to all shims to prevent improper resizing
marmite22 pushed a commit to elucidat/mediaelement that referenced this pull request Dec 16, 2016
Add "mejs-shim" class to all shims to prevent improper resizing
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