Commits

commits@gajim.org

17632 discussions

[Git][gajim/gajim][master] Refactor NewConfirmationDialog
by Philipp Hörist 26 Mar '19

26 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 21aa352a by Philipp Hörist at 2019-03-26T19:19:50Z Refactor NewConfirmationDialog - - - - - 3 changed files: - gajim/common/const.py - gajim/gtk/dialogs.py - gajim/gtk/themes.py Changes: ===================================== gajim/common/const.py ===================================== @@ -6,9 +6,6 @@ from gajim.common.i18n import _ Option = namedtuple('Option', 'kind label type value name callback data desc enabledif props') Option.__new__.__defaults__ = (None,) * len(Option._fields) # type: ignore -DialogButton = namedtuple('DialogButton', 'text callback action') -DialogButton.__new__.__defaults__ = (None, None) # type: ignore - EncryptionData = namedtuple('EncryptionData', 'additional_data') EncryptionData.__new__.__defaults__ = (None,) # type: ignore ===================================== gajim/gtk/dialogs.py ===================================== @@ -12,6 +12,8 @@ # You should have received a copy of the GNU General Public License # along with Gajim. If not, see <http://www.gnu.org/licenses/>. +from collections import namedtuple + from gi.repository import Gtk from gajim.common import app @@ -22,6 +24,36 @@ from gajim.gtk.util import get_builder from gajim.gtk.util import load_icon +class DialogButton(namedtuple('DialogButton', ('response text callback args ' + 'kwargs action is_default'))): + @classmethod + def make(cls, type_=None, **kwargs): + # Defaults + default_kwargs = { + 'response': None, + 'text': None, + 'callback': None, + 'args': [], + 'kwargs': {}, + 'action': None, + 'is_default': False + } + + if type_ is not None: + if type_ == 'OK': + default_kwargs['response'] = Gtk.ResponseType.OK + default_kwargs['text'] = 'OK' + + elif type_ == 'Cancel': + default_kwargs['response'] = Gtk.ResponseType.CANCEL + default_kwargs['text'] = _('Cancel') + else: + raise ValueError('Unknown button type: %s ' % type_) + + default_kwargs.update(kwargs) + return cls(**default_kwargs) + + class HigDialog(Gtk.MessageDialog): def __init__(self, parent, type_, buttons, pritext, sectext, on_response_ok=None, on_response_cancel=None, on_response_yes=None, @@ -952,18 +984,22 @@ class ChangePasswordDialog(Gtk.Dialog): class NewConfirmationDialog(Gtk.MessageDialog): - def __init__(self, text, sec_text, buttons, transient_for=None): + def __init__(self, title, text, sec_text, buttons, transient_for=None): Gtk.MessageDialog.__init__(self, + title=title, + text=text, transient_for=transient_for, - message_type=Gtk.MessageType.QUESTION, - text=text) + message_type=Gtk.MessageType.QUESTION) - self._buttons = buttons + self._buttons = {} - for response, button in buttons.items(): - self.add_button(button.text, response) + for button in buttons: + self._buttons[button.response] = button + self.add_button(button.text, button.response) + if button.is_default: + self.set_default_response(button.response) if button.action is not None: - widget = self.get_widget_for_response(response) + widget = self.get_widget_for_response(button.response) widget.get_style_context().add_class(button.action.value) self.format_secondary_markup(sec_text) @@ -972,7 +1008,7 @@ class NewConfirmationDialog(Gtk.MessageDialog): self.run() - def _on_response(self, dialog, response): + def _on_response(self, _dialog, response): if response == Gtk.ResponseType.DELETE_EVENT: # Look if DELETE_EVENT is mapped to another response response = self._buttons.get(response, None) @@ -986,7 +1022,7 @@ class NewConfirmationDialog(Gtk.MessageDialog): return if button.callback is not None: - button.callback() + button.callback(*button.args, **button.kwargs) self.destroy() ===================================== gajim/gtk/themes.py ===================================== @@ -24,10 +24,10 @@ from gajim.common import app from gajim.common.nec import NetworkEvent from gajim.common.i18n import _ from gajim.common.const import StyleAttr -from gajim.common.const import DialogButton from gajim.common.const import ButtonAction from gajim.gtk.dialogs import ErrorDialog +from gajim.gtk.dialogs import DialogButton from gajim.gtk.dialogs import NewConfirmationDialog from gajim.gtk.util import get_builder @@ -320,17 +320,17 @@ class Themes(Gtk.ApplicationWindow): self._add_option_button.set_sensitive(False) self._clear_options() - buttons = { - Gtk.ResponseType.CANCEL: DialogButton('Keep Theme'), - Gtk.ResponseType.OK: DialogButton('Delete', - _remove_theme, - ButtonAction.DESTRUCTIVE), - } - - NewConfirmationDialog('Delete Theme', - 'Do you want to permanently delete this theme?', - buttons, - transient_for=self) + NewConfirmationDialog( + _('Delete'), + _('Delete Theme'), + _('Do you want to permanently ' + 'delete this theme?'), + [DialogButton.make('Cancel'), + DialogButton.make('OK', + text=_('Delete'), + callback=_remove_theme, + action=ButtonAction.DESTRUCTIVE)], + transient_for=self) @staticmethod def _on_destroy(*args): View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/21aa352a344948d076f0fc033aa021312c… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/21aa352a344948d076f0fc033aa021312c… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][gajim_1.1] Remove reimport
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch gajim_1.1 at gajim / gajim Commits: 366b3df2 by Philipp Hörist at 2019-03-25T23:03:36Z Remove reimport - - - - - 1 changed file: - gajim/plugins/gui.py Changes: ===================================== gajim/plugins/gui.py ===================================== @@ -42,7 +42,6 @@ from gajim.common.helpers import launch_browser_mailer from gajim.plugins.helpers import log_calls from gajim.plugins.helpers import GajimPluginActivateException from gajim.plugins.plugins_i18n import _ -from gajim.common.exceptions import PluginsystemError @unique View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/366b3df235cdb1cdd54dc909ea5c2900af… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/366b3df235cdb1cdd54dc909ea5c2900af… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][gajim_1.1] 9 commits: Add server blabber.im to servers.json
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch gajim_1.1 at gajim / gajim Commits: 40cf9488 by Dominion at 2019-03-25T22:15:26Z Add server blabber.im to servers.json - - - - - 64bda9cf by Daniel Brötzmann at 2019-03-25T22:19:11Z Flatpak: Disable install from ZIP - - - - - 838b04d6 by Daniel Brötzmann at 2019-03-25T22:24:33Z Improve dark theme colors - - - - - 23c08892 by Philipp Hörist at 2019-03-25T22:30:10Z Add method to determine window manager - - - - - 2f5d00d1 by Philipp Hörist at 2019-03-25T22:46:53Z Refactor saving roster position - Dont save roster position on Wayland - - - - - 0e9422ae by Philipp Hörist at 2019-03-25T22:51:46Z Refactor restoring roster position - - - - - 669671e0 by Philipp Hörist at 2019-03-25T22:52:37Z Move imports to the top - - - - - da5ffe4b by Philipp Hörist at 2019-03-25T22:53:05Z Correctly get the total screen geometry get_root_window() does not work on Wayland Fixes #9637 - - - - - f9872d3a by Philipp Hörist at 2019-03-25T22:59:07Z Update ChangeLog - - - - - 12 changed files: - ChangeLog - gajim/common/app.py - gajim/common/const.py - gajim/common/helpers.py - gajim/data/gui/plugins_window.ui - gajim/data/other/servers.json - gajim/data/style/default-dark.css - gajim/gtk/history.py - gajim/gtk/util.py - gajim/plugins/gui.py - gajim/roster_window.py - gajim/statusicon.py Changes: ===================================== ChangeLog ===================================== @@ -6,6 +6,9 @@ Gajim 1.1.3 (TBA) * #9573 Dont leak DNS query when connecting over proxy * #9578 Determine Windows version more reliably * #9622 Fix an error while quitting Gajim + * #9637 Restore window size correctly on wayland + * Improve dark theme colors + * Fix access to GnuPG keys * Use UUID4 item ids for pubsub posts * Various smaller improvements ===================================== gajim/common/app.py ===================================== @@ -39,11 +39,13 @@ from distutils.version import LooseVersion as V from collections import namedtuple import nbxmpp +from gi.repository import Gdk import gajim from gajim.common import config as c_config from gajim.common import configpaths from gajim.common import ged as ged_module +from gajim.common.const import Display from gajim.common.contacts import LegacyContactsAPI from gajim.common.events import Events from gajim.common.types import NetworkEventsControllerT # pylint: disable=unused-import @@ -193,6 +195,18 @@ def is_installed(dependency): def is_flatpak(): return gajim.IS_FLATPAK +def is_display(display): + # XWayland reports as Display X11, so try with env var + is_wayland = os.environ.get('XDG_SESSION_TYPE') == 'wayland' + if is_wayland and display == Display.WAYLAND: + return True + + default = Gdk.Display.get_default() + if default is None: + log('gajim').warning('Could not determine window manager') + return False + return default.__class__.__name__ == display.value + def disable_dependency(dependency): _dependencies[dependency] = False ===================================== gajim/common/const.py ===================================== @@ -193,6 +193,13 @@ class SyncThreshold(IntEnum): return str(self.value) +class Display(Enum): + X11 = 'X11Display' + WAYLAND = 'GdkWaylandDisplay' + WIN32 = 'GdkWin32Display' + QUARTZ = 'GdkQuartzDisplay' + + ACTIVITIES = { 'doing_chores': { 'category': _('Doing Chores'), ===================================== gajim/common/helpers.py ===================================== @@ -42,6 +42,7 @@ import logging import json import shutil import collections +from io import StringIO from datetime import datetime, timedelta from distutils.version import LooseVersion as V from encodings.punycode import punycode_encode @@ -51,11 +52,17 @@ import nbxmpp from nbxmpp.stringprepare import nameprep import precis_i18n.codec # pylint: disable=unused-import +from gajim.common import app from gajim.common import caps_cache from gajim.common import configpaths from gajim.common.i18n import Q_ from gajim.common.i18n import _ from gajim.common.i18n import ngettext +from gajim.common.const import Display + +if app.is_installed('PYCURL'): + import pycurl + log = logging.getLogger('gajim.c.helpers') @@ -552,12 +559,6 @@ def datetime_tuple(timestamp): tim = tim.timetuple() return tim - -from gajim.common import app -if app.is_installed('PYCURL'): - import pycurl - from io import StringIO - def convert_bytes(string): suffix = '' # IEC standard says KiB = 1024 bytes KB = 1000 bytes @@ -1534,3 +1535,14 @@ class AdditionalDataDict(collections.UserDict): del _dict[key] except KeyError: return + + +def save_roster_position(window): + if not app.config.get('save-roster-position'): + return + if app.is_display(Display.WAYLAND): + return + x_pos, y_pos = window.get_position() + log.debug('Save roster position: %s %s', x_pos, y_pos) + app.config.set('roster_x-position', x_pos) + app.config.set('roster_y-position', y_pos) ===================================== gajim/data/gui/plugins_window.ui ===================================== @@ -1,5 +1,5 @@ <?xml version="1.0" encoding="UTF-8"?> - + <interface> <requires lib="gtk+" version="3.20"/> <object class="GtkWindow" id="plugins_window"> @@ -13,6 +13,9 @@ <property name="type_hint">dialog</property> <signal name="destroy" handler="on_plugins_window_destroy" swapped="no"/> <signal name="key-press-event" handler="on_key_press_event" swapped="no"/> + <child type="titlebar"> + <placeholder/> + </child> <child> <object class="GtkNotebook" id="plugins_notebook"> <property name="visible">True</property> @@ -232,6 +235,7 @@ <property name="wrap">True</property> <property name="wrap_mode">word-char</property> <property name="selectable">True</property> + <property name="xalign">0</property> </object> <packing> <property name="expand">False</property> @@ -375,9 +379,6 @@ </child> </object> </child> - <child type="titlebar"> - <placeholder/> - </child> </object> <object class="GtkTextBuffer" id="textbuffer1"> <property name="text" translatable="yes">Plug-in decription should be displayed here. This text will be erased during PluginsWindow initialization.</property> ===================================== gajim/data/other/servers.json ===================================== @@ -2,6 +2,7 @@ "servers":[ "0nl1ne.at", "404.city", + "blabber.im", "brauchen.info", "chatme.im", "comm.unicate.me", ===================================== gajim/data/style/default-dark.css ===================================== @@ -1,11 +1,11 @@ .gajim-incoming-nickname { - color: rgb(164, 0, 0) + color: rgb(207, 49, 47) } .gajim-outgoing-nickname { - color: rgb(52, 101, 164) + color: rgb(38, 139, 210) } .gajim-url { - color: rgb(117, 80, 123) + color: rgb(53, 132, 228) } .gajim-highlight-message { color: rgb(245, 121, 0) @@ -15,4 +15,4 @@ } .gajim-status-message { color: rgb(115, 210, 22) - } \ No newline at end of file + } ===================================== gajim/gtk/history.py ===================================== @@ -33,7 +33,9 @@ from gajim.common import app from gajim.common import helpers from gajim.common import exceptions from gajim.common.i18n import _ -from gajim.common.const import ShowConstant, KindConstant +from gajim.common.const import ShowConstant +from gajim.common.const import KindConstant +from gajim.common.const import StyleAttr from gajim import conversation_textview @@ -79,7 +81,9 @@ class HistoryWindow: account, used_in_history_window=True) scrolledwindow.add(self.history_textview.tv) self.history_buffer = self.history_textview.tv.get_buffer() - self.history_buffer.create_tag('highlight', background='yellow') + highlight_color = app.css_config.get_value( + '.gajim-highlight-message', StyleAttr.COLOR) + self.history_buffer.create_tag('highlight', background=highlight_color) self.history_buffer.create_tag('invisible', invisible=True) self.checkbutton = xml.get_object('log_history_checkbutton') self.show_status_checkbutton = xml.get_object('show_status_checkbutton') ===================================== gajim/gtk/util.py ===================================== @@ -30,6 +30,7 @@ from gajim.common import app from gajim.common import configpaths from gajim.common import i18n from gajim.common.i18n import _ +from gajim.common.const import Display _icon_theme = Gtk.IconTheme.get_default() _icon_theme.append_search_path(configpaths.get('ICONS')) @@ -117,11 +118,17 @@ def get_iconset_name_for(name: str) -> str: def get_total_screen_geometry() -> Tuple[int, int]: - screen = Gdk.Screen.get_default() - window = Gdk.Screen.get_root_window(screen) - width, height = window.get_width(), window.get_height() - log.debug('Get screen geometry: %s %s', width, height) - return width, height + total_width = 0 + total_height = 0 + display = Gdk.Display.get_default() + monitors = display.get_n_monitors() + for num in range(0, monitors): + monitor = display.get_monitor(num) + geometry = monitor.get_geometry() + total_width += geometry.width + total_height = max(total_height, geometry.height) + log.debug('Get screen geometry: %s %s', total_width, total_height) + return total_width, total_height def resize_window(window: Gtk.Window, width: int, height: int) -> None: @@ -155,6 +162,16 @@ def move_window(window: Gtk.Window, pos_x: int, pos_y: int) -> None: window.move(pos_x, pos_y) +def restore_roster_position(window): + if not app.config.get('save-roster-position'): + return + if app.is_display(Display.WAYLAND): + return + move_window(window, + app.config.get('roster_x-position'), + app.config.get('roster_y-position')) + + def get_completion_liststore(entry: Gtk.Entry) -> Gtk.ListStore: """ Create a completion model for entry widget completion list consists of ===================================== gajim/plugins/gui.py ===================================== @@ -36,6 +36,9 @@ from gajim.gtk.dialogs import YesNoDialog from gajim.gtk.filechoosers import ArchiveChooserDialog from gajim.common import app from gajim.common import configpaths +from gajim.common.exceptions import PluginsystemError +from gajim.common.helpers import launch_browser_mailer + from gajim.plugins.helpers import log_calls from gajim.plugins.helpers import GajimPluginActivateException from gajim.plugins.plugins_i18n import _ @@ -63,14 +66,21 @@ class PluginsWindow: widgets_to_extract = ('plugins_notebook', 'plugin_name_label', 'plugin_version_label', 'plugin_authors_label', - 'plugin_homepage_linkbutton', 'uninstall_plugin_button', - 'configure_plugin_button', 'installed_plugins_treeview', - 'available_text', 'available_text_label') + 'plugin_homepage_linkbutton', 'install_plugin_button', + 'uninstall_plugin_button', 'configure_plugin_button', + 'installed_plugins_treeview', 'available_text', + 'available_text_label') for widget_name in widgets_to_extract: setattr(self, widget_name, builder.get_object(widget_name)) self.plugin_description_textview = builder.get_object('description') + + # Disable 'Install from ZIP' for Flatpak installs + if app.is_flatpak(): + self.install_plugin_button.set_tooltip_text( + _('Click to view Gajim\'s wiki page on how to install plugins in Flatpak.')) + self.installed_plugins_model = Gtk.ListStore(object, str, bool, bool, GdkPixbuf.Pixbuf) self.installed_plugins_treeview.set_model(self.installed_plugins_model) @@ -237,6 +247,10 @@ class PluginsWindow: @log_calls('PluginsWindow') def on_install_plugin_button_clicked(self, widget): + if app.is_flatpak(): + launch_browser_mailer('url', 'https://dev.gajim.org/gajim/gajim/wikis/help/flathub') + return + def show_warn_dialog(): text = _('Archive is malformed') dialog = WarningDialog(text, '', transient_for=self.window) ===================================== gajim/roster_window.py ===================================== @@ -59,6 +59,7 @@ from gajim.common import helpers from gajim.common import idle from gajim.common.exceptions import GajimGeneralException from gajim.common import i18n +from gajim.common.helpers import save_roster_position from gajim.common.i18n import _ from gajim.common.const import PEPEventType, AvatarSize, StyleAttr from gajim.common.dbus import location @@ -83,6 +84,8 @@ from gajim.gtk.service_registration import ServiceRegistration from gajim.gtk.history import HistoryWindow from gajim.gtk.accounts import AccountsWindow +from gajim.gtk.util import restore_roster_position + log = logging.getLogger('gajim.roster') @@ -2407,11 +2410,7 @@ class RosterWindow: if not app.config.get('quit_on_roster_x_button') and ( (app.interface.systray_enabled and app.config.get('trayicon') != \ 'on_event') or app.config.get('allow_hide_roster')): - if app.config.get('save-roster-position'): - x, y = self.window.get_position() - log.debug('Save roster position (get_position): %s %s', x, y) - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(self.window) if os.name == 'nt' or app.config.get('hide_on_roster_x_button'): self.window.hide() else: @@ -2436,11 +2435,7 @@ class RosterWindow: # in case show_roster_on_start is False and roster is never shown # window.window is None if self.window.get_window() is not None: - if app.config.get('save-roster-position'): - x, y = self.window.get_window().get_root_origin() - log.debug('Save roster position (get_root_origin): %s %s', x, y) - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(self.window) width, height = self.window.get_size() app.config.set('roster_width', width) app.config.set('roster_height', height) @@ -5706,13 +5701,11 @@ class RosterWindow: if len(app.connections) < 2: # Do not merge accounts if only one exists self.regroup = False + gtkgui_helpers.resize_window(self.window, app.config.get('roster_width'), app.config.get('roster_height')) - if app.config.get('save-roster-position'): - gtkgui_helpers.move_window(self.window, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + restore_roster_position(self.window) self.popups_notification_height = 0 self.popup_notification_windows = [] ===================================== gajim/statusicon.py ===================================== @@ -30,6 +30,8 @@ from gajim import gtkgui_helpers from gajim.common import app from gajim.common import helpers from gajim.common.i18n import _ +from gajim.common.helpers import save_roster_position +from gajim.gtk.util import restore_roster_position from gajim.gtk.single_message import SingleMessageWindow @@ -371,18 +373,11 @@ class StatusIcon: # No pending events, so toggle visible/hidden for roster window if win.get_property('visible'): if win.get_property('has-toplevel-focus') or os.name == 'nt': - if app.config.get('save-roster-position'): - x, y = win.get_position() - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(win) win.hide() # else we hide it from VD that was visible in else: - if not win.get_property('visible'): - win.show_all() - if app.config.get('save-roster-position'): - gtkgui_helpers.move_window(win, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + win.show_all() + restore_roster_position(win) if not app.config.get('roster_window_skip_taskbar'): win.set_property('skip-taskbar-hint', False) win.present_with_time(Gtk.get_current_event_time()) @@ -394,11 +389,9 @@ class StatusIcon: if not event: return win = app.interface.roster.window - if not win.get_property('visible') and app.config.get( - 'save-roster-position'): - gtkgui_helpers.move_window(win, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + if not win.get_property('visible'): + # Needed if we are in one window mode + restore_roster_position(win) app.interface.handle_event(account, jid, event.type_) def on_middle_click(self): View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/65dc8ed89254a8bec6e8eb0eb31ca0640… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/65dc8ed89254a8bec6e8eb0eb31ca0640… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Correctly get the total screen geometry
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 1551ffa0 by Philipp Hörist at 2019-03-25T22:09:06Z Correctly get the total screen geometry get_root_window() does not work on Wayland Fixes #9637 - - - - - 1 changed file: - gajim/gtk/util.py Changes: ===================================== gajim/gtk/util.py ===================================== @@ -271,11 +271,17 @@ def get_available_iconsets(): def get_total_screen_geometry() -> Tuple[int, int]: - screen = Gdk.Screen.get_default() - window = Gdk.Screen.get_root_window(screen) - width, height = window.get_width(), window.get_height() - log.debug('Get screen geometry: %s %s', width, height) - return width, height + total_width = 0 + total_height = 0 + display = Gdk.Display.get_default() + monitors = display.get_n_monitors() + for num in range(0, monitors): + monitor = display.get_monitor(num) + geometry = monitor.get_geometry() + total_width += geometry.width + total_height = max(total_height, geometry.height) + log.debug('Get screen geometry: %s %s', total_width, total_height) + return total_width, total_height def resize_window(window: Gtk.Window, width: int, height: int) -> None: View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/1551ffa0b402f07a913b101c8421f1830e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/1551ffa0b402f07a913b101c8421f1830e… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Move imports to the top
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 82016886 by Philipp Hörist at 2019-03-25T21:26:55Z Move imports to the top - - - - - 1 changed file: - gajim/common/helpers.py Changes: ===================================== gajim/common/helpers.py ===================================== @@ -42,6 +42,7 @@ import logging import json import shutil import collections +from io import StringIO from datetime import datetime, timedelta from distutils.version import LooseVersion as V from encodings.punycode import punycode_encode @@ -60,6 +61,10 @@ from gajim.common.i18n import ngettext from gajim.common.const import ShowConstant from gajim.common.const import Display +if app.is_installed('PYCURL'): + import pycurl + + log = logging.getLogger('gajim.c.helpers') special_groups = (_('Transports'), _('Not in Roster'), _('Observers'), _('Groupchats')) @@ -558,12 +563,6 @@ def datetime_tuple(timestamp): tim = tim.timetuple() return tim - -from gajim.common import app -if app.is_installed('PYCURL'): - import pycurl - from io import StringIO - def convert_bytes(string): suffix = '' # IEC standard says KiB = 1024 bytes KB = 1000 bytes View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/82016886843c1c2b65dd592000913b75b9… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/82016886843c1c2b65dd592000913b75b9… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] 2 commits: Refactor saving roster position
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: a9efde13 by Philipp Hörist at 2019-03-25T21:20:19Z Refactor saving roster position - Dont save roster position on Wayland - - - - - 7a5d791b by Philipp Hörist at 2019-03-25T21:20:19Z Refactor restoring roster position - - - - - 4 changed files: - gajim/common/helpers.py - gajim/gtk/statusicon.py - gajim/gtk/util.py - gajim/roster_window.py Changes: ===================================== gajim/common/helpers.py ===================================== @@ -51,12 +51,14 @@ import nbxmpp from nbxmpp.stringprepare import nameprep import precis_i18n.codec # pylint: disable=unused-import +from gajim.common import app from gajim.common import caps_cache from gajim.common import configpaths from gajim.common.i18n import Q_ from gajim.common.i18n import _ from gajim.common.i18n import ngettext from gajim.common.const import ShowConstant +from gajim.common.const import Display log = logging.getLogger('gajim.c.helpers') @@ -1508,3 +1510,14 @@ class AdditionalDataDict(collections.UserDict): del _dict[key] except KeyError: return + + +def save_roster_position(window): + if not app.config.get('save-roster-position'): + return + if app.is_display(Display.WAYLAND): + return + x_pos, y_pos = window.get_position() + log.debug('Save roster position: %s %s', x_pos, y_pos) + app.config.set('roster_x-position', x_pos) + app.config.set('roster_y-position', y_pos) ===================================== gajim/gtk/statusicon.py ===================================== @@ -28,10 +28,11 @@ from gajim import dialogs from gajim.common import app from gajim.common import helpers from gajim.common.i18n import _ +from gajim.common.helpers import save_roster_position from gajim.gtk.util import get_builder from gajim.gtk.util import get_icon_name -from gajim.gtk.util import move_window +from gajim.gtk.util import restore_roster_position from gajim.gtk.single_message import SingleMessageWindow from gajim.gtk.tooltips import NotificationAreaTooltip @@ -375,18 +376,11 @@ class StatusIcon: # No pending events, so toggle visible/hidden for roster window if win.get_property('visible'): if win.get_property('has-toplevel-focus') or os.name == 'nt': - if app.config.get('save-roster-position'): - x, y = win.get_position() - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(win) win.hide() # else we hide it from VD that was visible in else: - if not win.get_property('visible'): - win.show_all() - if app.config.get('save-roster-position'): - move_window(win, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + win.show_all() + restore_roster_position(win) if not app.config.get('roster_window_skip_taskbar'): win.set_property('skip-taskbar-hint', False) win.present_with_time(Gtk.get_current_event_time()) @@ -398,11 +392,9 @@ class StatusIcon: if not event: return win = app.interface.roster.window - if not win.get_property('visible') and app.config.get( - 'save-roster-position'): - move_window(win, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + if not win.get_property('visible'): + # Needed if we are in one window mode + restore_roster_position(win) app.interface.handle_event(account, jid, event.type_) def on_middle_click(self): ===================================== gajim/gtk/util.py ===================================== @@ -40,6 +40,7 @@ from gajim.common.i18n import _ from gajim.common.const import MOODS from gajim.common.const import ACTIVITIES from gajim.common.const import LOCATION_DATA +from gajim.common.const import Display from gajim.gtk.const import GajimIconSet @@ -308,6 +309,16 @@ def move_window(window: Gtk.Window, pos_x: int, pos_y: int) -> None: window.move(pos_x, pos_y) +def restore_roster_position(window): + if not app.config.get('save-roster-position'): + return + if app.is_display(Display.WAYLAND): + return + move_window(window, + app.config.get('roster_x-position'), + app.config.get('roster_y-position')) + + def get_completion_liststore(entry: Gtk.Entry) -> Gtk.ListStore: """ Create a completion model for entry widget completion list consists of ===================================== gajim/roster_window.py ===================================== @@ -56,6 +56,7 @@ from gajim.common import helpers from gajim.common import idle from gajim.common.exceptions import GajimGeneralException from gajim.common import i18n +from gajim.common.helpers import save_roster_position from gajim.common.i18n import _ from gajim.common.const import PEPEventType, AvatarSize, StyleAttr from gajim.common.dbus import location @@ -83,7 +84,7 @@ from gajim.gtk.tooltips import RosterTooltip from gajim.gtk.adhoc_commands import CommandWindow from gajim.gtk.util import get_icon_name from gajim.gtk.util import resize_window -from gajim.gtk.util import move_window +from gajim.gtk.util import restore_roster_position from gajim.gtk.util import get_metacontact_surface from gajim.gtk.util import get_builder from gajim.gtk.util import set_urgency_hint @@ -2365,11 +2366,7 @@ class RosterWindow: if not app.config.get('quit_on_roster_x_button') and ( (app.interface.systray_enabled and app.config.get('trayicon') != \ 'on_event') or app.config.get('allow_hide_roster')): - if app.config.get('save-roster-position'): - x, y = self.window.get_position() - log.debug('Save roster position (get_position): %s %s', x, y) - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(self.window) if os.name == 'nt' or app.config.get('hide_on_roster_x_button'): self.window.hide() else: @@ -2394,11 +2391,7 @@ class RosterWindow: # in case show_roster_on_start is False and roster is never shown # window.window is None if self.window.get_window() is not None: - if app.config.get('save-roster-position'): - x, y = self.window.get_window().get_root_origin() - log.debug('Save roster position (get_root_origin): %s %s', x, y) - app.config.set('roster_x-position', x) - app.config.set('roster_y-position', y) + save_roster_position(self.window) width, height = self.window.get_size() app.config.set('roster_width', width) app.config.set('roster_height', height) @@ -5625,10 +5618,7 @@ class RosterWindow: resize_window(self.window, app.config.get('roster_width'), app.config.get('roster_height')) - if app.config.get('save-roster-position'): - move_window(self.window, - app.config.get('roster_x-position'), - app.config.get('roster_y-position')) + restore_roster_position(self.window) # Remove contact from roster when last event opened # { (contact, account): { backend: boolean } View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/b6e7f5c6d2ecfa6be6044e2bec58ed93f… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/compare/b6e7f5c6d2ecfa6be6044e2bec58ed93f… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Format OMEMO fingerprint in tooltip
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: b6e7f5c6 by Daniel Brötzmann at 2019-03-25T20:00:40Z Format OMEMO fingerprint in tooltip - - - - - 2 changed files: - gajim/conversation_textview.py - gajim/gtk/util.py Changes: ===================================== gajim/conversation_textview.py ===================================== @@ -49,6 +49,7 @@ from gajim.common.const import StyleAttr, Trust from gajim.gtk import util from gajim.gtk.util import load_icon from gajim.gtk.util import get_cursor +from gajim.gtk.util import format_fingerprint from gajim.gtk.emoji_data import emoji_pixbufs from gajim.gtk.emoji_data import is_emoji from gajim.gtk.emoji_data import get_emoji_pixbuf @@ -1159,7 +1160,8 @@ class ConversationTextview(GObject.GObject): icon, trust_tooltip, color = TRUST_SYMBOL_DATA[trust] tooltip = tooltip + '\n' + trust_tooltip if fingerprint is not None: - tooltip = tooltip + ' (' + fingerprint + ')' + fingerprint = format_fingerprint(fingerprint) + tooltip = tooltip + '\n' + fingerprint temp_mark = self._buffer.create_mark(None, iter_, True) self._buffer.insert(iter_, ' ') ===================================== gajim/gtk/util.py ===================================== @@ -23,6 +23,7 @@ from typing import Optional import os import sys import logging +import textwrap import xml.etree.ElementTree as ET from pathlib import Path from functools import wraps @@ -578,3 +579,13 @@ def format_location(location): 'tag': tag.capitalize(), 'text': text} return location_string.strip() + + +def format_fingerprint(fingerprint): + fplen = len(fingerprint) + wordsize = fplen // 8 + buf = '' + for w in range(0, fplen, wordsize): + buf += '{0} '.format(fingerprint[w:w + wordsize]) + buf = textwrap.fill(buf, width=36) + return buf.rstrip().upper() View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/b6e7f5c6d2ecfa6be6044e2bec58ed93fe… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/b6e7f5c6d2ecfa6be6044e2bec58ed93fe… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Add method to determine window manager
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 4acf36c9 by Philipp Hörist at 2019-03-25T19:11:53Z Add method to determine window manager - - - - - 2 changed files: - gajim/common/app.py - gajim/common/const.py Changes: ===================================== gajim/common/app.py ===================================== @@ -39,12 +39,14 @@ from distutils.version import LooseVersion as V from collections import namedtuple import nbxmpp +from gi.repository import Gdk import gajim from gajim.common import config as c_config from gajim.common import configpaths from gajim.common import ged as ged_module from gajim.common.i18n import LANG +from gajim.common.const import Display from gajim.common.contacts import LegacyContactsAPI from gajim.common.events import Events from gajim.common.types import NetworkEventsControllerT # pylint: disable=unused-import @@ -212,6 +214,18 @@ def is_installed(dependency): def is_flatpak(): return gajim.IS_FLATPAK +def is_display(display): + # XWayland reports as Display X11, so try with env var + is_wayland = os.environ.get('XDG_SESSION_TYPE') == 'wayland' + if is_wayland and display == Display.WAYLAND: + return True + + default = Gdk.Display.get_default() + if default is None: + log('gajim').warning('Could not determine window manager') + return False + return default.__class__.__name__ == display.value + def disable_dependency(dependency): _dependencies[dependency] = False ===================================== gajim/common/const.py ===================================== @@ -199,6 +199,7 @@ class MUCUser(IntEnum): AFFILIATION = 3 AFFILIATION_TEXT = 4 + @unique class Trust(IntEnum): UNTRUSTED = 0 @@ -206,6 +207,14 @@ class Trust(IntEnum): BLIND = 2 VERIFIED = 3 + +class Display(Enum): + X11 = 'X11Display' + WAYLAND = 'GdkWaylandDisplay' + WIN32 = 'GdkWin32Display' + QUARTZ = 'GdkQuartzDisplay' + + EME_MESSAGES = { 'urn:xmpp:otr:0': _('This message was encrypted with OTR ' View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/4acf36c97b79730b40ded2bbb050fd18eb… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/4acf36c97b79730b40ded2bbb050fd18eb… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Update look of Remove account dialog
by Philipp Hörist 25 Mar '19

25 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: f6e113c0 by Daniel Brötzmann at 2019-03-25T19:11:33Z Update look of Remove account dialog - - - - - 3 changed files: - gajim/config.py - gajim/data/gui/remove_account_window.ui - gajim/gtk/accounts.py Changes: ===================================== gajim/config.py ===================================== @@ -30,19 +30,6 @@ from gi.repository import Gtk from gi.repository import GObject -from gajim.common import app -from gajim.common import passwords -from gajim.common.i18n import _ - -from gajim import dialogs - -from gajim import gui_menu_builder - -from gajim.gtk.dialogs import ConfirmationDialog -from gajim.gtk.dialogs import ConfirmationDialogDoubleRadio -from gajim.gtk.dialogs import ErrorDialog -from gajim.gtk.util import get_builder - class FakeDataForm(Gtk.Table): """ @@ -94,131 +81,3 @@ class FakeDataForm(Gtk.Table): for name in self.entries: self.infos[name] = self.entries[name].get_text() return self.infos - -#---------- RemoveAccountWindow class -------------# -class RemoveAccountWindow: - """ - Ask for removing from gajim only or from gajim and server too and do - removing of the account given - """ - - def on_remove_account_window_destroy(self, widget): - if self.account in app.interface.instances: - del app.interface.instances[self.account]['remove_account'] - - def on_cancel_button_clicked(self, widget): - self.window.destroy() - - def __init__(self, account): - self.account = account - xml = get_builder('remove_account_window.ui') - self.window = xml.get_object('remove_account_window') - active_window = app.app.get_active_window() - self.window.set_transient_for(active_window) - self.remove_and_unregister_radiobutton = xml.get_object( - 'remove_and_unregister_radiobutton') - self.window.set_title(_('Removing %s account') % self.account) - xml.connect_signals(self) - self.window.show_all() - - def on_remove_button_clicked(self, widget): - def remove(): - if self.account in app.connections and \ - app.connections[self.account].connected and \ - not self.remove_and_unregister_radiobutton.get_active(): - # change status to offline only if we will not remove this JID from - # server - app.connections[self.account].change_status('offline', 'offline') - if self.remove_and_unregister_radiobutton.get_active(): - if not self.account in app.connections: - ErrorDialog( - _('Account is disabled'), - _('To unregister from a server, account must be ' - 'enabled.'), - transient_for=self.window) - return - if not app.connections[self.account].password: - def on_ok(passphrase, checked): - if passphrase == -1: - # We don't remove account cause we canceled pw window - return - app.connections[self.account].password = passphrase - app.connections[self.account].unregister_account( - self._on_remove_success) - - dialogs.PassphraseDialog( - _('Password Required'), - _('Enter your password for account %s') % self.account, - _('Save password'), ok_handler=on_ok, - transient_for=self.window) - return - app.connections[self.account].unregister_account( - self._on_remove_success) - else: - self._on_remove_success(True) - - if self.account in app.connections and \ - app.connections[self.account].connected: - ConfirmationDialog( - _('Account "%s" is connected to the server') % self.account, - _('If you remove it, the connection will be lost.'), - on_response_ok=remove, - transient_for=self.window) - else: - remove() - - def on_remove_responce_ok(self, is_checked): - if is_checked[0]: - self._on_remove_success(True) - - def _on_remove_success(self, res): - # action of unregistration has failed, we don't remove the account - # Error message is send by connect_and_auth() - if not res: - ConfirmationDialogDoubleRadio( - _('Connection to server %s failed') % self.account, - _('What would you like to do?'), - _('Remove only from Gajim'), - _('Don\'t remove anything. I\'ll try again later'), - on_response_ok=self.on_remove_responce_ok, is_modal=False, - transient_for=self.window) - return - # Close all opened windows - app.interface.roster.close_all(self.account, force=True) - if self.account in app.connections: - app.connections[self.account].disconnect(reconnect=False) - app.connections[self.account].cleanup() - del app.connections[self.account] - app.logger.remove_roster(app.get_jid_from_account(self.account)) - # Delete password must be before del_per() because it calls set_per() - # which would recreate the account with defaults values if not found - passwords.delete_password(self.account) - app.config.del_per('accounts', self.account) - del app.interface.instances[self.account] - if self.account in app.nicks: - del app.interface.minimized_controls[self.account] - del app.nicks[self.account] - del app.block_signed_in_notifications[self.account] - del app.groups[self.account] - app.contacts.remove_account(self.account) - del app.gc_connected[self.account] - del app.automatic_rooms[self.account] - del app.to_be_removed[self.account] - del app.newly_added[self.account] - del app.sleeper_state[self.account] - del app.last_message_time[self.account] - del app.status_before_autoaway[self.account] - del app.gajim_optional_features[self.account] - del app.caps_hash[self.account] - if len(app.connections) >= 2: # Do not merge accounts if only one exists - app.interface.roster.regroup = app.config.get('mergeaccounts') - else: - app.interface.roster.regroup = False - app.interface.roster.setup_and_draw_roster() - app.app.remove_account_actions(self.account) - gui_menu_builder.build_accounts_menu() - - window = app.get_app_window('AccountsWindow') - if window is not None: - window.remove_account(self.account) - self.window.destroy() ===================================== gajim/data/gui/remove_account_window.ui ===================================== @@ -1,119 +1,115 @@ <?xml version="1.0" encoding="UTF-8"?> - + <interface> - <requires lib="gtk+" version="3.12"/> - <object class="GtkImage" id="image1"> - <property name="visible">True</property> - <property name="can_focus">False</property> - <property name="stock">gtk-delete</property> - </object> + <requires lib="gtk+" version="3.20"/> <object class="GtkWindow" id="remove_account_window"> <property name="can_focus">False</property> - <property name="border_width">6</property> + <property name="border_width">18</property> <property name="type_hint">dialog</property> <signal name="destroy" handler="on_remove_account_window_destroy" swapped="no"/> <child> - <object class="GtkBox" id="vbox69"> + <placeholder/> + </child> + <child> + <object class="GtkBox"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="border_width">3</property> <property name="orientation">vertical</property> - <property name="spacing">6</property> + <property name="spacing">18</property> <child> - <object class="GtkBox" id="hbox2951"> + <object class="GtkLabel"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="spacing">6</property> + <property name="halign">start</property> + <property name="label" translatable="yes">What do you want to do?</property> + <property name="use_markup">True</property> + <style> + <class name="large-header"/> + </style> + </object> + <packing> + <property name="expand">False</property> + <property name="fill">True</property> + <property name="position">0</property> + </packing> + </child> + <child> + <object class="GtkBox"> + <property name="visible">True</property> + <property name="can_focus">False</property> + <property name="spacing">12</property> <child> - <object class="GtkImage" id="image498"> + <object class="GtkImage"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="stock">gtk-dialog-question</property> + <property name="icon_name">user-trash</property> <property name="icon_size">6</property> </object> <packing> <property name="expand">False</property> - <property name="fill">False</property> + <property name="fill">True</property> <property name="position">0</property> </packing> </child> <child> - <object class="GtkFrame" id="frame30"> + <object class="GtkBox"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="label_xalign">0</property> - <property name="shadow_type">none</property> + <property name="orientation">vertical</property> + <property name="spacing">6</property> <child> - <object class="GtkBox" id="vbox1"> + <object class="GtkRadioButton" id="remove_only_radiobutton"> + <property name="label" translatable="yes">Remove account _only from Gajim</property> <property name="visible">True</property> - <property name="can_focus">False</property> - <property name="margin_left">12</property> - <property name="border_width">6</property> - <property name="orientation">vertical</property> - <property name="spacing">6</property> - <child> - <object class="GtkRadioButton" id="remove_only_radiobutton"> - <property name="label" translatable="yes">Remove account _only from Gajim</property> - <property name="visible">True</property> - <property name="can_focus">True</property> - <property name="receives_default">False</property> - <property name="use_underline">True</property> - <property name="xalign">0</property> - <property name="active">True</property> - <property name="draw_indicator">True</property> - </object> - <packing> - <property name="expand">False</property> - <property name="fill">False</property> - <property name="position">0</property> - </packing> - </child> - <child> - <object class="GtkRadioButton" id="remove_and_unregister_radiobutton"> - <property name="label" translatable="yes">Remove account from Gajim and from _server</property> - <property name="visible">True</property> - <property name="can_focus">True</property> - <property name="receives_default">False</property> - <property name="use_underline">True</property> - <property name="xalign">0</property> - <property name="draw_indicator">True</property> - <property name="group">remove_only_radiobutton</property> - </object> - <packing> - <property name="expand">False</property> - <property name="fill">False</property> - <property name="position">1</property> - </packing> - </child> + <property name="can_focus">True</property> + <property name="receives_default">False</property> + <property name="halign">start</property> + <property name="use_underline">True</property> + <property name="active">True</property> + <property name="draw_indicator">True</property> </object> + <packing> + <property name="expand">False</property> + <property name="fill">True</property> + <property name="position">0</property> + </packing> </child> - <child type="label"> - <object class="GtkLabel" id="label242"> + <child> + <object class="GtkRadioButton" id="remove_and_unregister_radiobutton"> + <property name="label" translatable="yes">Remove account from Gajim and from _server</property> <property name="visible">True</property> - <property name="can_focus">False</property> - <property name="label" translatable="yes"><b>What do you want to do?</b></property> - <property name="use_markup">True</property> + <property name="can_focus">True</property> + <property name="receives_default">False</property> + <property name="halign">start</property> + <property name="use_underline">True</property> + <property name="active">True</property> + <property name="draw_indicator">True</property> + <property name="group">remove_only_radiobutton</property> </object> + <packing> + <property name="expand">False</property> + <property name="fill">False</property> + <property name="position">1</property> + </packing> </child> </object> <packing> <property name="expand">False</property> - <property name="fill">False</property> + <property name="fill">True</property> <property name="position">1</property> </packing> </child> </object> <packing> <property name="expand">False</property> - <property name="fill">False</property> - <property name="position">0</property> + <property name="fill">True</property> + <property name="position">1</property> </packing> </child> <child> - <object class="GtkButtonBox" id="hbuttonbox16"> + <object class="GtkButtonBox"> <property name="visible">True</property> <property name="can_focus">False</property> - <property name="border_width">6</property> <property name="spacing">12</property> <property name="layout_style">end</property> <child> @@ -139,9 +135,11 @@ <property name="can_focus">True</property> <property name="can_default">True</property> <property name="receives_default">False</property> - <property name="image">image1</property> <property name="use_underline">True</property> <signal name="clicked" handler="on_remove_button_clicked" swapped="no"/> + <style> + <class name="destructive-action"/> + </style> </object> <packing> <property name="expand">False</property> @@ -153,7 +151,7 @@ <packing> <property name="expand">False</property> <property name="fill">False</property> - <property name="position">1</property> + <property name="position">2</property> </packing> </child> </object> ===================================== gajim/gtk/accounts.py ===================================== @@ -30,12 +30,14 @@ from gajim.common.const import OptionKind from gajim.common.const import OptionType from gajim import gui_menu_builder -from gajim import config +from gajim.dialogs import PassphraseDialog from gajim.options_dialog import OptionsDialog from gajim.options_dialog import OptionsBox from gajim.gtk.dialogs import ConfirmationDialog +from gajim.gtk.dialogs import ConfirmationDialogDoubleRadio +from gajim.gtk.dialogs import ErrorDialog from gajim.gtk.dialogs import YesNoDialog from gajim.gtk.util import get_icon_name from gajim.gtk.util import get_builder @@ -220,7 +222,7 @@ class AccountsWindow(Gtk.ApplicationWindow): if account not in app.interface.instances: app.interface.instances[account] = {} app.interface.instances[account]['remove_account'] = \ - config.RemoveAccountWindow(account) + RemoveAccountWindow(account) if win_opened: ConfirmationDialog( _('You have opened chat in account %s') % account, @@ -811,3 +813,131 @@ class LoginDialog(OptionsDialog): savepass = app.config.get_per('accounts', self.account, 'savepass') if not savepass: passwords.delete_password(self.account) + + +class RemoveAccountWindow: + """ + Ask whether to remove from gajim only or both from gajim and the server, + then remove the account given + """ + + def on_remove_account_window_destroy(self, widget): + if self.account in app.interface.instances: + del app.interface.instances[self.account]['remove_account'] + + def on_cancel_button_clicked(self, widget): + self._ui.remove_account_window.destroy() + + def __init__(self, account): + self.account = account + self._ui = get_builder('remove_account_window.ui') + active_window = app.app.get_active_window() + self._ui.remove_account_window.set_transient_for(active_window) + self._ui.remove_account_window.set_title(_('Removing account %s') % self.account) + self._ui.connect_signals(self) + self._ui.remove_account_window.show_all() + + def on_remove_button_clicked(self, widget): + def remove(): + if self.account in app.connections and \ + app.connections[self.account].connected and \ + not self._ui.remove_and_unregister_radiobutton.get_active(): + # change status to offline only if we will not remove this JID from + # server + app.connections[self.account].change_status('offline', 'offline') + if self._ui.remove_and_unregister_radiobutton.get_active(): + if not self.account in app.connections: + ErrorDialog( + _('Account is disabled'), + _('To unregister from a server, the account must be ' + 'enabled.'), + transient_for=self._ui.remove_account_window) + return + if not app.connections[self.account].password: + def on_ok(passphrase, checked): + if passphrase == -1: + # We don't remove account cause we canceled pw window + return + app.connections[self.account].password = passphrase + app.connections[self.account].unregister_account( + self._on_remove_success) + + PassphraseDialog( + _('Password required'), + _('Enter your password for account %s') % self.account, + _('Save password'), ok_handler=on_ok, + transient_for=self._ui.remove_account_window) + return + app.connections[self.account].unregister_account( + self._on_remove_success) + else: + self._on_remove_success(True) + + if self.account in app.connections and \ + app.connections[self.account].connected: + ConfirmationDialog( + _('Account "%s" is connected to the server') % self.account, + _('If you remove it, the connection will be lost.'), + on_response_ok=remove, + transient_for=self._ui.remove_account_window) + else: + remove() + + def on_remove_response_ok(self, is_checked): + if is_checked[0]: + self._on_remove_success(True) + + def _on_remove_success(self, res): + # action of unregistration has failed, we don't remove the account + # Error message is send by connect_and_auth() + if not res: + ConfirmationDialogDoubleRadio( + _('Connection to server %s failed') % self.account, + _('What would you like to do?'), + _('Remove only from Gajim'), + _('Don\'t remove anything. I\'ll try again later'), + on_response_ok=self.on_remove_response_ok, is_modal=False, + transient_for=self._ui.remove_account_window) + return + # Close all opened windows + app.interface.roster.close_all(self.account, force=True) + if self.account in app.connections: + app.connections[self.account].disconnect(reconnect=False) + app.connections[self.account].cleanup() + del app.connections[self.account] + app.logger.remove_roster(app.get_jid_from_account(self.account)) + # Delete password must be before del_per() because it calls set_per() + # which would recreate the account with defaults values if not found + passwords.delete_password(self.account) + app.config.del_per('accounts', self.account) + del app.interface.instances[self.account] + if self.account in app.nicks: + del app.interface.minimized_controls[self.account] + del app.nicks[self.account] + del app.block_signed_in_notifications[self.account] + del app.groups[self.account] + app.contacts.remove_account(self.account) + del app.gc_connected[self.account] + del app.automatic_rooms[self.account] + del app.to_be_removed[self.account] + del app.newly_added[self.account] + del app.sleeper_state[self.account] + del app.last_message_time[self.account] + del app.status_before_autoaway[self.account] + del app.gajim_optional_features[self.account] + del app.caps_hash[self.account] + if len(app.connections) >= 2: # Do not merge accounts if only one exists + app.interface.roster.regroup = app.config.get('mergeaccounts') + else: + app.interface.roster.regroup = False + app.interface.roster.setup_and_draw_roster() + app.app.remove_account_actions(self.account) + gui_menu_builder.build_accounts_menu() + + window = app.get_app_window('AccountsWindow') + if window is not None: + window.remove_account(self.account) + self._ui.remove_account_window.destroy() + + def destroy(self): + self._ui.remove_account_window.destroy() View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f6e113c0b685fe866f79949fac184a632e… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/f6e113c0b685fe866f79949fac184a632e… You're receiving this email because of your account on dev.gajim.org.

1 0

[Git][gajim/gajim][master] Improve dark theme colors
by Philipp Hörist 24 Mar '19

24 Mar '19

Philipp Hörist pushed to branch master at gajim / gajim Commits: 52111cd1 by Daniel Brötzmann at 2019-03-24T14:33:37Z Improve dark theme colors - - - - - 2 changed files: - gajim/data/style/default-dark.css - gajim/gtk/history.py Changes: ===================================== gajim/data/style/default-dark.css ===================================== @@ -1,11 +1,11 @@ .gajim-incoming-nickname { - color: rgb(164, 0, 0) + color: rgb(207, 49, 47) } .gajim-outgoing-nickname { - color: rgb(52, 101, 164) + color: rgb(38, 139, 210) } .gajim-url { - color: rgb(117, 80, 123) + color: rgb(53, 132, 228) } .gajim-highlight-message { color: rgb(245, 121, 0) @@ -27,4 +27,4 @@ } .gajim-state-paused { color: rgb(0, 0, 205) - } \ No newline at end of file + } ===================================== gajim/gtk/history.py ===================================== @@ -36,6 +36,7 @@ from gajim.common import exceptions from gajim.common.i18n import _ from gajim.common.const import ShowConstant from gajim.common.const import KindConstant +from gajim.common.const import StyleAttr from gajim import conversation_textview @@ -85,7 +86,9 @@ class HistoryWindow(Gtk.ApplicationWindow): account, used_in_history_window=True) self._ui.scrolledwindow.add(self.history_textview.tv) self.history_buffer = self.history_textview.tv.get_buffer() - self.history_buffer.create_tag('highlight', background='yellow') + highlight_color = app.css_config.get_value( + '.gajim-highlight-message', StyleAttr.COLOR) + self.history_buffer.create_tag('highlight', background=highlight_color) self.history_buffer.create_tag('invisible', invisible=True) self.clearing_search = False View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/52111cd1b353f02c33682e73ac6bf99448… -- View it on GitLab: https://dev.gajim.org/gajim/gajim/commit/52111cd1b353f02c33682e73ac6bf99448… You're receiving this email because of your account on dev.gajim.org.

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits