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

Fix graphics fillStyle instead of lineStyle #6226

Merged
merged 2 commits into from Nov 18, 2019
Merged

Conversation

ivanpopelyshev
Copy link
Collaborator

@ivanpopelyshev ivanpopelyshev commented Nov 16, 2019

Another bug again, found by @eXponenta

Bugged: https://www.pixiplayground.com/#/edit/QF2RXuO21iDTwvxGntVBr
Fixed: https://www.pixiplayground.com/#/edit/yhKcbj73S9RHy1ajD0tdX

I've added test on it. With our previous tests everything should be fine, but I'll make a pass on pixijs examples anyway.

Also it fixes our typings.

@ivanpopelyshev ivanpopelyshev changed the title fix for batch size=0 + test Fix graphics with batch size=0 Nov 16, 2019
@ivanpopelyshev ivanpopelyshev changed the title Fix graphics with batch size=0 Fix graphics fillStyle instead of lineStyle Nov 16, 2019
@ivanpopelyshev ivanpopelyshev added the 🕷 Bug Verified that it’s actually a legit bug that exists in the current release. label Nov 16, 2019
@ivanpopelyshev
Copy link
Collaborator Author

OK, its not fine, we have several copies of BatchPart in typings. Any ideas?

@ivanpopelyshev
Copy link
Collaborator Author

Should be fine now.

@codecov-io
Copy link

codecov-io commented Nov 16, 2019

Codecov Report

Merging #6226 into dev will increase coverage by 0.01%.
The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #6226      +/-   ##
==========================================
+ Coverage   76.69%   76.71%   +0.01%     
==========================================
  Files         202      202              
  Lines       10240    10238       -2     
==========================================
  Hits         7854     7854              
+ Misses       2386     2384       -2
Impacted Files Coverage Δ
packages/graphics/src/utils/BatchPart.js 100% <ø> (ø) ⬆️
packages/graphics/src/utils/index.js 100% <ø> (ø) ⬆️
packages/graphics/src/GraphicsGeometry.js 76.04% <80%> (+0.39%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c19dcbc...45c3065. Read the comment docs.

@bigtimebuddy bigtimebuddy added this to the v5.2.1 milestone Nov 18, 2019
@bigtimebuddy bigtimebuddy merged commit 00e1e12 into dev Nov 18, 2019
@bigtimebuddy bigtimebuddy deleted the dev-graphics-again branch November 18, 2019 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕷 Bug Verified that it’s actually a legit bug that exists in the current release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants