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



commit 6a4652bd35e23a4ae4ec3078be380ed09858e0ee
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

 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 ae9eb01b1..bc20b6982 100644
--- a/panels/display/cc-display-config-dbus.c
+++ b/panels/display/cc-display-config-dbus.c
@@ -795,6 +795,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]