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

2.0.0 #62

Merged
merged 1 commit into from
Aug 7, 2015
Merged

2.0.0 #62

merged 1 commit into from
Aug 7, 2015

Conversation

jonathantneal
Copy link
Owner

  • Added: UMD pattern and svg4everybody method to activate shim
  • Added: Grunt build process
  • Added: Option to remove legacy code from Grunt
  • Added: Option to customize PNG fallback from browser
  • Updated: SVG External Content feature-deficient browser detection
  • Updated: Exclusive activation on '' children of ''
  • Updated: Preservation of existing viewbox attribute for SVGs
  • Updated: Preservation of existing width and height attributes for PNG fallbacks
  • Updated: Many performance improvements for caching, AJAX, and polling

}

var uses = document.getElementsByTagName('use');
var requestAnimationFrame = window.requestAnimationFrame || window.setTimeout;
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where would I save them in the script?

@Rendez
Copy link

Rendez commented Aug 3, 2015

I think we should consider #45. The min version should be part of dist.

@Rendez
Copy link

Rendez commented Aug 4, 2015

Looking great @jonathantneal. Is it working on all the IEs so far?

The only thing is that seems like you created files within dist with legacy=false. Is no IE8 compat. the way we want this plugin to be by default? I think perhaps we should build both? People using bower include dist files directly in their dependencies.

@shaneog
Copy link

shaneog commented Aug 7, 2015

It would be great to get this merged. Is there anything else holding up the release?

@jonathantneal
Copy link
Owner Author

@shaneog, no. I pinged my coworkers yesterday to review it, and they only recommended I change the name of a few new APIs. Today I pinged my IRC friends and it came back with all thumbs up. It’s 10:30AM where I am. Pending no further changes, I’ll handle the merge, tweet, etc. on my lunch break.

@shaneog
Copy link

shaneog commented Aug 7, 2015

Thanks @jonathantneal.
I only ask since I have been using this branch to gain IE Edge support and the various changes (force pushes) have caused some build issues (i.e. cannot checkout that revision).

Added: UMD pattern and `svg4everybody` method to activate shim
Added: Grunt build process
Added: Option to remove legacy code from Grunt
Added: Option to manually shim SVG or External Content from the browser
Added: Option to customize PNG fallback from the browser
Updated: SVG External Content feature-deficient browser detection
Updated: Exclusive activation on '<use>' children of '<svg>'
Updated: Preservation of existing `viewbox` attribute for SVGs
Updated: Preservation of existing `width` and `height` attributes for PNG fallbacks
Updated: Many performance improvements for caching, AJAX, and polling
jonathantneal added a commit that referenced this pull request Aug 7, 2015
@jonathantneal jonathantneal merged commit 243335b into master Aug 7, 2015
@jonathantneal jonathantneal deleted the feature/2.0.0 branch August 7, 2015 17:46
@Rendez
Copy link

Rendez commented Aug 7, 2015

I had some comments sorry I was too busy at work. I'll add them later and before you tag the release :) cheers!

}

var polyfill = 'polyfill' in opts ? opts.polyfill : LEGACY_SUPPORT ? (
nosvg || /\bEdge\/12\b|\bMSIE [1-8]\b|\bTrident\/[567]\b|\bVersion\/7.0 Safari\b/.test(navigator.userAgent) || (navigator.userAgent.match(/AppleWebKit\/(\d+)/) || [])[1] < 537
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There duplication on this area, which could be reduced in bytes if saved in a variable.

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.

5 participants