Merge pull request #119 from MightyCreak/pylint

Extract dialogs and widgets into their own modules

Author: MightyCreak

.github/workflows/main.yml

@@ -23,7 +23,7 @@ jobs:
     - name: Pylint
       uses: cclauss/GitHub-Action-for-pylint@master
       with:
-        args: "pylint src/utils.py src/vcs/"
+        args: "pylint src/vcs/ src/dialogs.py src/preferences.py src/resources.py src/utils.py src/widgets.py"
 
   meson-build-test:
     runs-on: ubuntu-latest

.pylintrc

@@ -100,6 +100,7 @@ disable=raw-checker-failed,
         too-many-arguments,
         too-many-branches,
         too-many-instance-attributes,
+        too-many-lines,
         too-many-locals,
         too-many-nested-blocks,
         too-many-statements,

CHANGELOG.md

@@ -12,7 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added a flatpak job in the CI
 
 ### Changed
-- main.py slimmed down by about 1000 lines
+- main.py slimmed down by about 5000 lines
+- The new widgets.py is a bit fat though (~4000 lines)
 
 ### Fixed
 

src/dialogs.py

@@ -0,0 +1,194 @@
+# Diffuse: a graphical tool for merging and comparing text files.
+#
+# Copyright (C) 2019 Derrick Moser <[email protected]>
+# Copyright (C) 2021 Romain Failliot <[email protected]>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+import os
+
+# pylint: disable=wrong-import-position
+import gi
+gi.require_version('GObject', '2.0')
+gi.require_version('Gtk', '3.0')
+from gi.repository import GObject, Gtk
+# pylint: enable=wrong-import-position
+
+from diffuse import constants
+from diffuse import utils
+
+# the about dialog
+class AboutDialog(Gtk.AboutDialog):
+    def __init__(self):
+        Gtk.AboutDialog.__init__(self)
+        self.set_logo_icon_name('io.github.mightycreak.Diffuse')
+        self.set_program_name(constants.APP_NAME)
+        self.set_version(constants.VERSION)
+        self.set_comments(_('Diffuse is a graphical tool for merging and comparing text files.'))
+        self.set_copyright(constants.COPYRIGHT)
+        self.set_website(constants.WEBSITE)
+        self.set_authors([ 'Derrick Moser <[email protected]>',
+                           'Romain Failliot <[email protected]>' ])
+        self.set_translator_credits(_('translator-credits'))
+        license_text = [
+            constants.APP_NAME + ' ' + constants.VERSION + '\n\n',
+            constants.COPYRIGHT + '\n\n',
+            _('''This program is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2 of the License, or
+(at your option) any later version.
+
+This program is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+GNU General Public License for more details.
+
+You should have received a copy of the GNU General Public License along
+with this program; if not, write to the Free Software Foundation, Inc.,
+51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.''') ]
+        self.set_license(''.join(license_text))
+
+# custom dialogue for picking files with widgets for specifying the encoding
+# and revision
+class FileChooserDialog(Gtk.FileChooserDialog):
+    # record last chosen folder so the file chooser can start at a more useful
+    # location for empty panes
+    last_chosen_folder = os.path.realpath(os.curdir)
+
+    def __current_folder_changed_cb(self, widget):
+        FileChooserDialog.last_chosen_folder = widget.get_current_folder()
+
+    def __init__(self, title, parent, prefs, action, accept, rev=False):
+        Gtk.FileChooserDialog.__init__(self, title=title, parent=parent, action=action)
+        self.add_button(Gtk.STOCK_CANCEL, Gtk.ResponseType.CANCEL)
+        self.add_button(accept, Gtk.ResponseType.OK)
+        self.prefs = prefs
+        hbox = Gtk.Box.new(Gtk.Orientation.HORIZONTAL, 0)
+        hbox.set_border_width(5)
+        label = Gtk.Label.new(_('Encoding: '))
+        hbox.pack_start(label, False, False, 0)
+        label.show()
+        self.encoding = entry = utils.EncodingMenu(
+            prefs,
+            action in [ Gtk.FileChooserAction.OPEN, Gtk.FileChooserAction.SELECT_FOLDER ])
+        hbox.pack_start(entry, False, False, 5)
+        entry.show()
+        if rev:
+            self.revision = entry = Gtk.Entry.new()
+            hbox.pack_end(entry, False, False, 0)
+            entry.show()
+            label = Gtk.Label.new(_('Revision: '))
+            hbox.pack_end(label, False, False, 0)
+            label.show()
+
+        self.vbox.pack_start(hbox, False, False, 0) # pylint: disable=no-member
+        hbox.show()
+        self.set_current_folder(self.last_chosen_folder)
+        self.connect('current-folder-changed', self.__current_folder_changed_cb)
+
+    def set_encoding(self, encoding):
+        self.encoding.set_text(encoding)
+
+    def get_encoding(self):
+        return self.encoding.get_text()
+
+    def get_revision(self):
+        return self.revision.get_text()
+
+    def get_filename(self):
+        # convert from UTF-8 string to unicode
+        return Gtk.FileChooserDialog.get_filename(self) # pylint: disable=no-member
+
+# dialogue used to search for text
+class NumericDialog(Gtk.Dialog):
+    def __init__(self, parent, title, text, val, lower, upper, step=1, page=0):
+        Gtk.Dialog.__init__(self, title=title, parent=parent, destroy_with_parent=True)
+        self.add_button(Gtk.STOCK_CANCEL, Gtk.ResponseType.REJECT)
+        self.add_button(Gtk.STOCK_OK, Gtk.ResponseType.ACCEPT)
+
+        vbox = Gtk.Box.new(Gtk.Orientation.VERTICAL, 0)
+        vbox.set_border_width(10)
+
+        hbox = Gtk.Box.new(Gtk.Orientation.HORIZONTAL, 0)
+        label = Gtk.Label.new(text)
+        hbox.pack_start(label, False, False, 0)
+        label.show()
+        adj = Gtk.Adjustment.new(val, lower, upper, step, page, 0)
+        self.button = button = Gtk.SpinButton.new(adj, 1.0, 0)
+        button.connect('activate', self.button_cb)
+        hbox.pack_start(button, True, True, 0)
+        button.show()
+
+        vbox.pack_start(hbox, True, True, 0)
+        hbox.show()
+
+        self.vbox.pack_start(vbox, False, False, 0) # pylint: disable=no-member
+        vbox.show()
+
+    def button_cb(self, widget):
+        self.response(Gtk.ResponseType.ACCEPT)
+
+# dialogue used to search for text
+class SearchDialog(Gtk.Dialog):
+    def __init__(self, parent, pattern=None, history=None):
+        Gtk.Dialog.__init__(self, title=_('Find...'), parent=parent, destroy_with_parent=True)
+        self.add_button(Gtk.STOCK_CANCEL, Gtk.ResponseType.REJECT)
+        self.add_button(Gtk.STOCK_OK, Gtk.ResponseType.ACCEPT)
+
+        vbox = Gtk.Box.new(Gtk.Orientation.VERTICAL, 0)
+        vbox.set_border_width(10)
+
+        hbox = Gtk.Box.new(Gtk.Orientation.HORIZONTAL, 0)
+        label = Gtk.Label.new(_('Search For: '))
+        hbox.pack_start(label, False, False, 0)
+        label.show()
+        combo = Gtk.ComboBoxText.new_with_entry()
+        self.entry = combo.get_child()
+        self.entry.connect('activate', self.entry_cb)
+
+        if pattern is not None:
+            self.entry.set_text(pattern)
+
+        if history is not None:
+            completion = Gtk.EntryCompletion.new()
+            liststore = Gtk.ListStore(GObject.TYPE_STRING)
+            completion.set_model(liststore)
+            completion.set_text_column(0)
+            for h in history:
+                liststore.append([h])
+                combo.append_text(h)
+            self.entry.set_completion(completion)
+
+        hbox.pack_start(combo, True, True, 0)
+        combo.show()
+        vbox.pack_start(hbox, False, False, 0)
+        hbox.show()
+
+        button = Gtk.CheckButton.new_with_mnemonic(_('Match Case'))
+        self.match_case_button = button
+        vbox.pack_start(button, False, False, 0)
+        button.show()
+
+        button = Gtk.CheckButton.new_with_mnemonic(_('Search Backwards'))
+        self.backwards_button = button
+        vbox.pack_start(button, False, False, 0)
+        button.show()
+
+        self.vbox.pack_start(vbox, False, False, 0) # pylint: disable=no-member
+        vbox.show()
+
+    # callback used when the Enter key is pressed
+    def entry_cb(self, widget):
+        self.response(Gtk.ResponseType.ACCEPT)