1j01/textual-paint
1
1
Fork 0
mirror of https://github.com/1j01/textual-paint.git synced 2025-01-02 20:04:36 +03:00
Code Issues Packages Projects Releases Wiki Activity

Implement Edit > Clear Selection (Delete)

Browse Source
This commit is contained in:
Isaiah Odhner 2023-04-21 16:19:07 -04:00
parent 61d3db4714
commit e98931e8ef
1 changed files with 18 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
paint.py
View File

@ -947,6 +947,7 @@ class PaintApp(App):
("shift+ctrl+z", "redo", _("Repeat")), ("shift+ctrl+z", "redo", _("Repeat")),
("ctrl+y", "redo", _("Repeat")), ("ctrl+y", "redo", _("Repeat")),
("f4", "redo", _("Repeat")), ("f4", "redo", _("Repeat")),
("delete", "clear_selection", _("Clear Selection")),
("ctrl+pageup", "normal_size", _("Normal Size")), ("ctrl+pageup", "normal_size", _("Normal Size")),
("ctrl+pagedown", "large_size", _("Large Size")), ("ctrl+pagedown", "large_size", _("Large Size")),
# action_toggle_dark is built in to App # action_toggle_dark is built in to App
@ -1070,6 +1071,18 @@ class PaintApp(App):
self.image.bg[y][x] = bg_color self.image.bg[y][x] = bg_color
self.image.fg[y][x] = fg_color self.image.fg[y][x] = fg_color
def erase_region(self, region: Region) -> None:
# Time to go undercover as an eraser. 🥸
# TODO: just add a parameter to stamp_char.
# Momentarily masquerading makes me mildly mad.
original_tool = self.selected_tool
self.selected_tool = Tool.eraser
sel = self.image.selection
for x in range(sel.region.width):
for y in range(sel.region.height):
self.stamp_char(x + sel.region.x, y + sel.region.y)
self.selected_tool = original_tool
def action_undo(self) -> None: def action_undo(self) -> None:
self.meld_selection() self.meld_selection()
if len(self.undos) > 0: if len(self.undos) > 0:
@ -1374,8 +1387,6 @@ class PaintApp(App):
self.warning_message_box(_("Paint"), "Not implemented.", "ok") self.warning_message_box(_("Paint"), "Not implemented.", "ok")
def action_paste(self) -> None: def action_paste(self) -> None:
self.warning_message_box(_("Paint"), "Not implemented.", "ok") self.warning_message_box(_("Paint"), "Not implemented.", "ok")
def action_clear_selection(self) -> None:
self.warning_message_box(_("Paint"), "Not implemented.", "ok")
def action_select_all(self) -> None: def action_select_all(self) -> None:
self.warning_message_box(_("Paint"), "Not implemented.", "ok") self.warning_message_box(_("Paint"), "Not implemented.", "ok")
def action_copy_to(self) -> None: def action_copy_to(self) -> None:
@ -1624,14 +1635,7 @@ class PaintApp(App):
self.redos = [] self.redos = []
self.undos.append(action) self.undos.append(action)
self.image.selection.copy_from_document(self.image) self.image.selection.copy_from_document(self.image)
# Time to go undercover as an eraser. 🥸 self.erase_region(sel.region)
# TODO: just add a parameter to stamp_char.
# Momentarily masquerading makes me mildly mad.
self.selected_tool = Tool.eraser
for x in range(sel.region.width):
for y in range(sel.region.height):
self.stamp_char(x + sel.region.x, y + sel.region.y)
self.selected_tool = Tool.select
# TODO: use two regions, for the cut out and the paste in, once melded. # TODO: use two regions, for the cut out and the paste in, once melded.
# I could maybe give Action a sub_action property, and use it for the melding. # I could maybe give Action a sub_action property, and use it for the melding.
# Or I could make it use a list of regions. # Or I could make it use a list of regions.
@ -1762,10 +1766,13 @@ class PaintApp(App):
self.canvas.refresh_scaled_region(region) self.canvas.refresh_scaled_region(region)
self.selection_drag_offset = None self.selection_drag_offset = None
def clear_selection(self) -> None: def action_clear_selection(self) -> None:
"""Delete the selection and its contents.""" """Delete the selection and its contents."""
if self.image.selection: if self.image.selection:
region = self.image.selection.region region = self.image.selection.region
if not self.image.selection.contained_image:
# It hasn't been cut out yet, so we need to erase it.
self.erase_region(region)
self.image.selection = None self.image.selection = None
self.canvas.refresh_scaled_region(region) self.canvas.refresh_scaled_region(region)
self.selection_drag_offset = None self.selection_drag_offset = None