Set translate/modify entry css

data/se.sjoerd.Graphs.appdata.xml.in.in

@@ -78,6 +78,8 @@
           <li>Importing column data is now more robust, supporting expressions as data points</li>
           <li>Pinch and ctrl+scroll to zoom gestures are now supported on the canvas</li>
           <li>The canvas can now be panned using two fingers on a touch screen, as well as with the middle mouse button</li>
+          <li>Toasts now show an "Open Location" button when saving data, bringing you to the saved file location</li>
+          <li>Translate/multiply entries now get a red CSS when input value is invalid, while the corresponding button is disabled</li>
         </ul>
         <p>Changed behaviour:</p>
         <ul>

data/whats_new

@@ -23,6 +23,8 @@
           <li>Importing column data is now more robust, supporting expressions as data points</li>
           <li>Pinch and ctrl+scroll to zoom gestures are now supported on the canvas</li>
           <li>The canvas can now be panned using two fingers on a touch screen, as well as with the middle mouse button</li>
+          <li>Toasts now show an "Open Location" button when saving data, bringing you to the saved file location</li>
+          <li>Translate/multiply entries now get a red CSS when input value is invalid, while the corresponding button is disabled</li>
         </ul>
         <p>Changed behaviour:</p>
         <ul>

src/application.py

@@ -10,7 +10,8 @@
 
 from gi.repository import GLib, GObject, Gio, Graphs, Gtk
 
-from graphs import actions, file_import, file_io, migrate, styles, ui
+from graphs import (actions, file_import, file_io, migrate, styles, ui,
+                    utilities)
 from graphs.data import Data
 
 from matplotlib import font_manager
@@ -221,6 +222,14 @@ def do_activate(self):
             stack_switcher.set_hexpand("true")
             window.get_stack_switcher_box().prepend(stack_switcher)
             window.set_title(self.props.name)
+            actions = ["multiply_x", "multiply_y", "translate_x",
+                       "translate_y"]
+            entries = [getattr(window, f"get_{action}_entry")()
+                       for action in actions]
+            buttons = [getattr(window, f"get_{action}_button")()
+                       for action in actions]
+            for entry, button in zip(entries, buttons):
+                entry.connect("notify::text", self.set_entry_css, button)
             self.set_window(window)
             controller = Gtk.EventControllerKey.new()
             controller.connect("key-pressed", self.on_key_press_event)
@@ -234,3 +243,16 @@ def do_activate(self):
                 "notify::items", ui.enable_axes_actions, self)
             ui.enable_axes_actions(self, None, self)
             window.present()
+
+    def set_entry_css(self, entry, _string, button):
+        try:
+            value = utilities.string_to_float(entry.get_text())
+            if value is None and entry.get_text() != "":
+                entry.add_css_class("error")
+                button.set_sensitive(False)
+                return
+            entry.remove_css_class("error")
+            button.set_sensitive(True)
+        except (ValueError, TypeError):
+            entry.add_css_class("error")
+            button.set_sensitive(False)

src/canvas.py

@@ -316,7 +316,7 @@ def _redraw(self, *_args):
             # Set tick where requested, as long as that axis is not occupied
             # and visible
             if (params[f"xtick.{directions[0]}"]
-                    or params[f"ytick.{directions[1]}"]) :
+                    or params[f"ytick.{directions[1]}"]):
                 axis.tick_params(which=ticks, **{
                     direction: (draw_frame and not visible_axes[i]
                                 or direction in directions)

src/window.vala

@@ -54,6 +54,18 @@ namespace Graphs {
         [GtkChild]
         public unowned Entry multiply_y_entry { get; }
 
+        [GtkChild]
+        public unowned Button translate_x_button { get; }
+
+        [GtkChild]
+        public unowned Button translate_y_button { get; }
+
+        [GtkChild]
+        public unowned Button multiply_x_button { get; }
+
+        [GtkChild]
+        public unowned Button multiply_y_button { get; }
+
         [GtkChild]
         public unowned ListBox item_list { get; }