Use <Alt> to invert the value of the "outline" option (#46)

Tools: pencil, line, curve
maoschanz · Nov 3, 2021 · abf12f6 · abf12f6
1 parent a285687
commit abf12f6
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 3 deletions.
diff --git a/src/tools/abstract_tool.py b/src/tools/abstract_tool.py
@@ -94,8 +94,7 @@ def get_settings(self):
 
 	def update_modifier_state(self, event_state):
 		modifier_keys = []
-		if (event_state & Gdk.ModifierType.CONTROL_MASK) == Gdk.ModifierType.CONTROL_MASK:
-			modifier_keys.append("CTRL")
+		# CONTROL_MASK can't be used, because it already has an effect app-wide.
 		if (event_state & Gdk.ModifierType.SHIFT_MASK) == Gdk.ModifierType.SHIFT_MASK:
 			modifier_keys.append("SHIFT")
 		if (event_state & Gdk.ModifierType.MOD1_MASK) == Gdk.ModifierType.MOD1_MASK:

diff --git a/src/tools/classic_tools/tool_arc.py b/src/tools/classic_tools/tool_arc.py
@@ -76,6 +76,10 @@ def get_edition_status(self):
 	def on_press_on_area(self, event, surface, event_x, event_y):
 		self.set_common_values(event.button, event_x, event_y)
 
+		self.update_modifier_state(event.state)
+		if "ALT" in self._modifier_keys:
+			self._use_outline = not self._use_outline
+
 	def on_motion_on_area(self, event, surface, event_x, event_y, render=True):
 		cairo_context = self.get_context()
 		if self._1st_segment is None:

diff --git a/src/tools/classic_tools/tool_line.py b/src/tools/classic_tools/tool_line.py
@@ -78,7 +78,10 @@ def on_press_on_area(self, event, surface, event_x, event_y):
 		self.set_common_values(event.button, event_x, event_y)
 
 		self.update_modifier_state(event.state)
-		self._ortholock = self._ortholock or "SHIFT" in self._modifier_keys
+		if "SHIFT" in self._modifier_keys:
+			self._ortholock = not self._ortholock
+		if "ALT" in self._modifier_keys:
+			self._use_outline = not self._use_outline
 
 	def on_motion_on_area(self, event, surface, event_x, event_y, render=True):
 		if render:

diff --git a/src/tools/classic_tools/tool_pencil.py b/src/tools/classic_tools/tool_pencil.py
@@ -67,6 +67,10 @@ def on_press_on_area(self, event, surface, event_x, event_y):
 		self.set_common_values(event.button, event_x, event_y)
 		self._path = None
 
+		self.update_modifier_state(event.state)
+		if "ALT" in self._modifier_keys:
+			self._use_outline = not self._use_outline
+
 	def _add_point(self, event_x, event_y):
 		cairo_context = self.get_context()
 		if self._path is None: