mirror/aseprite
1
0
Fork 0
mirror of https://github.com/aseprite/aseprite.git synced 2024-10-03 21:46:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix clearing/cutting several tiles at the same time from the tileset

Browse Source
This commit is contained in:
David Capello 2020-10-26 18:45:21 -03:00
parent 2d0227e8b2
commit 394b8bb2aa
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/app/ui/color_bar.cpp
View File

@ -1011,8 +1011,14 @@ app::Color ColorBar::onPaletteViewGetBackgroundIndex()
return getBgColor();
}
void ColorBar::onTilesViewClearTiles(const doc::PalettePicks& picks)
void ColorBar::onTilesViewClearTiles(const doc::PalettePicks& _picks)
{
// Copy the collection of selected tiles because in case that the
// user want to delete a range of tiles (several tiles at the same
// time), after the first cmd::RemoveTile() is executed this
// collection (_picks) will be modified and we'll lost the other
// selected tiles to remove.
doc::PalettePicks picks = _picks;
try {
ContextWriter writer(UIContext::instance(), 500);
Sprite* sprite = writer.sprite();
@ -1021,7 +1027,7 @@ void ColorBar::onTilesViewClearTiles(const doc::PalettePicks& picks)
auto tileset = m_tilesView.tileset();
Tx tx(writer.context(), "Clear Tiles", ModifyDocument);
for (doc::tile_index ti=0; ti<picks.size(); ++ti) {
for (int ti=int(picks.size())-1; ti>=0; --ti) {
if (picks[ti])
tx(new cmd::RemoveTile(tileset, ti));
}