1j01/textual-paint
1
1
Fork 0
mirror of https://github.com/1j01/textual-paint.git synced 2024-11-28 10:03:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

Don't invert the inner textbox border, that's too visually confusing

Browse Source
This commit is contained in:
Isaiah Odhner 2023-04-25 18:31:50 -04:00
parent 5ba8cbdbb8
commit ba7f8e6a0a
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@ -127,7 +127,7 @@ cat file.ans
## Known Issues
- Pressing Delete while using the Text tool deletes the whole text box. Similarly, Ctrl+A selects the canvas instead of the text in the text box.
- Text and Selection box borders are inside instead of outside (and have no dashes).
- Selection box border is inside instead of outside (and lacks dashes). For the text box, I hid the border because it was too visually confusing, but it should also have an outer border.
- The currently selected foreground (text) color is not displayed in the palette.
- Tools can't be cancelled with Escape or by pressing both mouse buttons.
- Help > Help Topics isn't very helpful.

2
paint.py
View File

@ -1043,7 +1043,7 @@ class Canvas(Widget):
if (
(self.magnifier_preview_region and magnifier_preview_region.contains(x, y) and (not inner_magnifier_preview_region.contains(x, y))) or
(self.select_preview_region and select_preview_region.contains(x, y) and (not inner_select_preview_region.contains(x, y))) or
(sel and selection_region.contains(x, y) and (not inner_selection_region.contains(x, y))) or
(sel and (not sel.textbox_mode) and selection_region.contains(x, y) and (not inner_selection_region.contains(x, y))) or
(sel and sel.textbox_mode and (
# offset_to_text_index(sel.text_selection_start) <=
# offset_to_text_index(Offset(x, y))