ui,popups: restrict widgets width, allow split words

Long paths or binary names can misalign the dialog.

In order to prevent this:
 - don't allow labels to grow more than the dialog's width.
 - if the labels text is larger than max_value, use zero-width space to
   allow to split words.

Author: gustavo-iniguez-goya

ui/opensnitch/dialogs/prompt.py

@@ -69,6 +69,7 @@ def __init__(self, parent=None, appicon=None):
 
         self._width = self.width()
         self._height = self.height()
+        self.reset_widgets()
 
         dialog_geometry = self._cfg.getSettings("promptDialog/geometry")
         if dialog_geometry == QtCore.QByteArray:
@@ -155,6 +156,22 @@ def showEvent(self, event):
         self.adjust_size()
         self.move_popup()
 
+    def reset_widgets(self):
+        # Don't allow labels to grow more than the dialog's width.
+        # This can happen if the path or the binary name is too large.
+
+        self.appNameLabel.setMaximumWidth(self._width-5)
+        self.appDescriptionLabel.setMaximumWidth(self._width-5)
+        self.appPathLabel.setMaximumWidth(self._width-5)
+        self.argsLabel.setMaximumWidth(self._width-5)
+        self.messageLabel.setMaximumWidth(self._width-5)
+
+        self.appNameLabel.setText("")
+        self.appDescriptionLabel.setText("")
+        self.appPathLabel.setText("")
+        self.argsLabel.setText("")
+        self.messageLabel.setText("")
+
     def adjust_size(self):
         if self._width is None or self._height is None:
             self._width = self.width()
@@ -208,6 +225,8 @@ def _set_elide_text(self, widget, text, max_size=64):
         widget.setText(text)
 
     def promptUser(self, connection, is_local, peer):
+        self.reset_widgets()
+
         # one at a time
         with self._lock:
             # reset state
@@ -298,6 +317,7 @@ def _set_app_description(self, description):
             self.appDescriptionLabel.setVisible(False)
             self.appDescriptionLabel.setFixedHeight(0)
             self.appDescriptionLabel.setText("")
+            return
 
         self.appDescriptionLabel.setText(
             "".join(
@@ -322,12 +342,15 @@ def _set_app_path(self, app_name, app_args, con):
         else:
             self.appPathLabel.setVisible(False)
             self.appPathLabel.setText("")
+            return
 
         self.appPathLabel.setText(
             "".join(
                 filter(str.isprintable, self.appPathLabel.text())
             )
         )
+        if self.appPathLabel.width() >= self._width:
+            self.appPathLabel.setText("\u200b".join(self.appPathLabel.text()))
 
     def _set_app_args(self, app_name, app_args):
         # if the app name and the args are the same, there's no need to display
@@ -339,12 +362,15 @@ def _set_app_args(self, app_name, app_args):
         else:
             self.argsLabel.setVisible(False)
             self.argsLabel.setText("")
+            return
 
         self.argsLabel.setText(
             "".join(
                 filter(str.isprintable, self.argsLabel.text())
             )
         )
+        if self.argsLabel.width() >= self._width:
+            self.argsLabel.setText("\u200b".join(self.argsLabel.text()))
 
     def _set_default_target(self, combo, con, app_name, app_args):
         # set appimage as default target if the process path starts with
@@ -560,6 +586,9 @@ def _get_popup_message(self, app_name, con):
                 con.protocol.upper(),
                 con.dst_port)
 
+        if self.messageLabel.width() >= self._width:
+            self.messageLabel.setText("\u200b".join(self.messageLabel.text()))
+
         return "%s %s" % (message, msg_action)
 
     def _get_duration(self, duration_idx):