[geary: 2/3] ui: there're multiple actions with the field, use config val instead
- From: Michael Gratton <mjog src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [geary: 2/3] ui: there're multiple actions with the field, use config val instead
- Date: Mon, 25 Feb 2019 22:32:59 +0000 (UTC)
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]