Skip to content

Add correct zoom-scroll handling#671

Merged
ystrot merged 13 commits into
exyte:masterfrom
f3dm76:task/layer-animation-example
Apr 21, 2020
Merged

Add correct zoom-scroll handling#671
ystrot merged 13 commits into
exyte:masterfrom
f3dm76:task/layer-animation-example

Conversation

@f3dm76

@f3dm76 f3dm76 commented Feb 14, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread Source/views/MacawView.swift Outdated
class AnimationUtils {

class func layerForNodeRenderer(_ renderer: NodeRenderer, _ context: AnimationContext, animation: Animation, customBounds: Rect? = .none, shouldRenderContent: Bool = true) -> ShapeLayer {
class func layerForNodeRenderer(_ renderer: NodeRenderer, animation: Animation, customBounds: Rect? = .none, shouldRenderContent: Bool = true) -> ShapeLayer {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe we should try to use maximum line length equal to 120 symbols (for example)? Long lines are really uncomfortable to read and understand, especially here on Github.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would be great to add it to our SwiftLint 😉

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you agree - it takes about 5 mines to update rules for SwiftLint and maybe some time to format current issues.

@ystrot ystrot self-assigned this Apr 20, 2020
@ystrot ystrot added this to the 0.9.7 milestone Apr 20, 2020
@ystrot ystrot merged commit 15027f0 into exyte:master Apr 21, 2020
@f3dm76 f3dm76 deleted the task/layer-animation-example branch May 28, 2020 08:35
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.

3 participants