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

Transforms - mostly implemented #21

Open
wants to merge 130 commits into
base: master
Choose a base branch
from
Open

Transforms - mostly implemented #21

wants to merge 130 commits into from

Conversation

adamgit
Copy link

@adamgit adamgit commented Sep 3, 2012

Massive MASSIVE changes here (150 files changed!)

Generating this pull request so you can see what's changed, see what you think...

adamgit and others added 30 commits April 24, 2012 16:25
… important variable.

ALSO: disabled the class "SVGPathView", since it is UNDOCUMENTED and USED NOWHERE
… only represent the file / URL on disk, cleaned up the "export to CALayers" (more cleanup work left to do there), and upgraded the SVGKit library to compile cleanly on iOS 5.x
…ut fails the unit tests that were previously failing)
macros that were markers for adamgit but were breaking compile on other
computers
now only imports UIKit if target os is ios
… simpler, and reports Errors in a clearer manner

SVGPad demo project has been updated to report fatal errors with a popup
STRANGELY, some of our previously-working SVG's are now failing with an error that seems incorrect - bug in the refactoring?
Added some minor fixes to remove warnings that were in previous commit
fixed some more iOS references making sure they are only called when
building the iOS library. however an issue has come up regarding the
change from the SVGDocument to using SVGImage.

the entire SVGView class is under the assumption we are still using the
SVGDocument and SVGDocument+CA classes. throughout the class we refer
to a synthesised property called document that uses functions from the
now no longer existing SVGDocument. document.width, document.height
ect.  i attempted to change these to use the new class of SVGImage but
the new functions (svgheight, svgwidth act) are incompatible data
types. most likely the SVGView class will need a rewrite in order to
use the new SVGImage class
… BREAK SOME EXISTING APPS THAT DEPEND ON THE OLD INCORRECT IMPLEMENTATION
…, zero build warnings on OS X and zero on iOS
adamgit and others added 30 commits November 1, 2012 08:48
…s, and has two subclasses: one that is fast and easy to use, the other that supports Apple's CoreAnimations on EACH INDIVIDUAL SVG ELEMENT (NB: these two classes are separate because supporting CoreAnimation at a fine level of detail prevents some major features such as auto-redraw-at-highest-resolution)
ADDED: parser now records "all the XML namespaces I found while parsing, and their prefixes / tags" (useful for debugging, and you can use it for some DOM-manipulation)
CREATED: a helper class for shared methods that DOM specification implicitly requires, but which they didn't define
Conflicts:
	SVGKit-iOS.xcodeproj/project.pbxproj
(from stolzda) - Implemented Support for path > "Qq", fixed minor Bug
…5 bug - I've removed the "removeObserver" line even though it SHOULD NOT be removed
Added sample SVG for Q + ZM path element
ADDED: sample SVG that tests the Polyline parsing
…lly support submodules, way too many bugs in git here :( )
…lly support submodules, way too many bugs in git here :( )
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.

4 participants