[gnome-control-center/gnome-40] display-config: Do not invert order when constructing modes



commit aadf89b44257b64225c6b6ef1980255c4b95fb88
Author: Robert Mader <robert mader posteo de>
Date:   Sun Jan 2 20:55:15 2022 +0100

    display-config: Do not invert order when constructing modes
    
    Mutter sends modes in descending order of preference. By reverting
    the order via `g_list_prepend`, we get unintended side effects
    such as choosing the least preferred refresh rate by default (if
    the selected mode is not marked as preferred).
    
    Instead of adding complex logic in several places, make sure that
    the assumption of descending preference is kept by simply not
    inverting the order.
    
    Closes https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/1562
    
    (cherry picked from commit 6a4652bd35e23a4ae4ec3078be380ed09858e0ee)

 panels/display/cc-display-config-dbus.c | 2 ++
 1 file changed, 2 insertions(+)
---
diff --git a/panels/display/cc-display-config-dbus.c b/panels/display/cc-display-config-dbus.c
index cdc8f8aef..477f103d8 100644
--- a/panels/display/cc-display-config-dbus.c
+++ b/panels/display/cc-display-config-dbus.c
@@ -793,6 +793,8 @@ construct_modes (CcDisplayMonitorDBus *self,
       if (mode->flags & MODE_CURRENT)
         self->current_mode = CC_DISPLAY_MODE (mode);
     }
+
+  self->modes = g_list_reverse (self->modes);
 }
 
 static CcDisplayMonitorDBus *


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