Skip to content

fix to read multiple objectEffects#163

Closed
sngjuk wants to merge 7 commits intomeltingice:masterfrom
sngjuk:patch-2
Closed

fix to read multiple objectEffects#163
sngjuk wants to merge 7 commits intomeltingice:masterfrom
sngjuk:patch-2

Conversation

@sngjuk
Copy link
Copy Markdown
Contributor

@sngjuk sngjuk commented Jun 25, 2019

13dab09
objectEffects block id varies from lfx2 to lmfx with multiple layer effect(e.g. multiple strokes), add lmfx to object_effect.coffee for support, otherwise objectEffect layer is skipped.

Layer info block keys:
image lfx2 when single stroke. (line 2)
image lmfx when multiple stroke.

image single stroke info

image multiple stroke info

sngjuk added 2 commits June 25, 2019 18:13
as info block id varies from 'lfx2' to 'lmfx' with multiple layer effect, add 'lmfx' to support it.
otherwise, objectEffects is skipped.
@sngjuk sngjuk closed this Jun 26, 2019
@sngjuk sngjuk deleted the patch-2 branch June 26, 2019 01:39
@sngjuk sngjuk restored the patch-2 branch June 26, 2019 01:39
as info block id varies from 'lfx2' to 'lmfx' with multiple layer effect, add 'lmfx' to support it.
otherwise, objectEffects is skipped
@sngjuk sngjuk reopened this Jun 26, 2019
sngjuk added 2 commits June 26, 2019 10:49
commit for travis build test
as info block id varies from 'lfx2' to 'lmfx' with multiple layer effect(e.g. multiple stroke), add 'lmfx' key id to support it.
otherwise, objectEffects are skipped
@sngjuk sngjuk changed the title fix to read multiple object effect fix to read multiple objectEffect Jun 26, 2019
@sngjuk sngjuk changed the title fix to read multiple objectEffect fix to read multiple objectEffects Jun 26, 2019
sngjuk added 2 commits June 26, 2019 16:52
-`multiple_object_effects.coffee`: class for multiple object effect, `lmfx`.
-`info.coffee`: add multipleObjectEffects for `lmfx`
@sngjuk sngjuk closed this Jun 26, 2019
@sngjuk
Copy link
Copy Markdown
Contributor Author

sngjuk commented Jun 26, 2019

reopend #164

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.

1 participant