[geary: 2/3] ui: there're multiple actions with the field, use config val instead



commit 8f2e4b43bee517fd8839d5783e3dff31988a02e9
Author: Konstantin Kharlamov <Hi-Angel yandex ru>
Date:   Sun Feb 24 20:30:53 2019 +0300

    ui: there're multiple actions with the field, use config val instead
    
    There are two actions for ACTION_COMPOSE_AS_HTML, and as bug #263 shows,
    under some circumstances one of them may have wrong value.
    
    However the config supposed to always have correct value, so query that
    instead.
    
    Signed-off-by: Konstantin Kharlamov <Hi-Angel yandex ru>

 src/client/composer/composer-widget.vala | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
---
diff --git a/src/client/composer/composer-widget.vala b/src/client/composer/composer-widget.vala
index c4e43cef..8b2bf1d6 100644
--- a/src/client/composer/composer-widget.vala
+++ b/src/client/composer/composer-widget.vala
@@ -2258,9 +2258,7 @@ public class ComposerWidget : Gtk.EventBox, Geary.BaseInterface {
         // so the user can still select text with a link in it,
         // without the popover immediately appearing and raining on
         // their text selection parade.
-        if (this.pointer_url != null &&
-            this.composer_actions.get_action_state(ACTION_COMPOSE_AS_HTML)
-                .get_boolean()) {
+        if (this.pointer_url != null && this.config.compose_as_html) {
             Gdk.EventButton? button = (Gdk.EventButton) event;
             Gdk.Rectangle location = Gdk.Rectangle();
             location.x = (int) button.x;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]