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

Canadian flag support for IE and Edge #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Canadian flag support for IE and Edge #50

wants to merge 1 commit into from

Conversation

ScottKaye
Copy link
Contributor

@ScottKaye ScottKaye commented Jun 22, 2016

Hi reddit!

This PR is a fix for the Canadian flag in IE and Edge. It has the unfortunate side effect of introducing a Firefox-specific hack because of a few issues with its renderer, however, both IE and Edge are now supported with a crisper leaf displaying on Webkit.

Something like this would never be used in production, this is just a collection of hacks to get a nice-looking Canadian flag to appear in Firefox, Chrome, IE 10+, and Edge. Other browsers might work too, but I didn't test beyond those four.

This has the unfortunate side effect of introducing a Firefox-specific hack because of a few issues with the renderer.  On the bright side, both IE and Edge are now supported, and a crisper leaf is displayed for Webkit-based renderers.
@manrajgrover
Copy link
Owner

@ScottKaye This is great but I would have to find an alternative to test it before merging. If you can share some screenshots, it would help too.

@ScottKaye
Copy link
Contributor Author

Should look identical to the existing one, but I can take some screen shots if that would help.

@manrajgrover
Copy link
Owner

@ScottKaye Will give me more confidence. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants