Closed hunkim98 closed 1 year ago
It seems to be an essential and interesting brush tool. I'm excited and looking forward to this feature!
Thank you all for your comments! I have solved some issues that you have raised! Unsolved issues can be discussed later on another PR!
By the we could also talk about some PR conventions maintainers could agree on such as using Pn rule when reviewing. We could discuss this when we all meet together!🚀
🚀 [Related Issue: #28 ]
Preview
https://user-images.githubusercontent.com/57612141/236849972-dc7642f8-14f2-4888-bd65-de05b1f0c724.mov
Changes
Mark selected pixels when brush tool is select
[🎨Component] Modify GridLayer and InteractionLayer to mark selected pixels
selectingArea
andselectedArea
for marking selected pixelsselectingArea
is set in Interaction Layer.selectingArea
transforms into aselectedArea
, which is the area in the pixel grid that was included in theselectingArea
. After it is transformed, theselectingArea
becomesnull
.selectingArea
or aselectedArea
the lines are rendered in the grid layer through the methodrenderSelection
[🔗Other] Remove render function calls in the methods of Grid Layers
render
method is directly called from theEditor
class. If therender
method is called in other layers, it is difficult to track, and difficult to draw additional drawings on that specific canvas context when needed.Notes
Next Up?