Package: xserver-xfree86
Version: 4.3.0-7
Severity: normal
Tags: patch

xserver-xfree86.config contains the following code:

...

if [ "$RET" = "true" ]; then
  db_subst xserver-xfree86/config/monitor/selection-method choices "Medium, 
Advanced"
  db_subst xserver-xfree86/config/monitor/selection-method default "Medium"
else # not an LCD
  db_subst xserver-xfree86/config/monitor/selection-method choices "Simple, 
Medium, Advanced"
  db_subst xserver-xfree86/config/monitor/selection-method default "Simple"
fi

auto_answer db_input "$(priority_ceil $PRIORITY)" 
xserver-xfree86/config/monitor/selection-method "Advanced"

...

Thus default answer for xserver-xfree86/config/monitor/selection-method
is "Simple" or "Medium" (which is good for non-technical users), but if
debconf actually asks the question, the default shown answer is
hardcoded to "Advanced". Since users tend to accept all defaults they
see (even after a help screen was shown describing what the selections
mean), too often they select this default and are faced with questions
about frequences they can't understand. And even can't go back at that
point.

So it seems better to show a "silly-safe" choise by default. I.e. the
same as is set in the case when question is not asked.

--- xserver-xfree86.config.orig 2004-04-05 16:02:07.000000000 +0400
+++ xserver-xfree86.config      2004-04-05 16:06:21.000000000 +0400
@@ -1568,13 +1568,14 @@
 MONITOR_IS_LCD="$RET"
 if [ "$RET" = "true" ]; then
   db_subst xserver-xfree86/config/monitor/selection-method choices "Medium, 
Advanced"
-  db_subst xserver-xfree86/config/monitor/selection-method default "Medium"
+  DEFAULT="Medium"
 else # not an LCD
   db_subst xserver-xfree86/config/monitor/selection-method choices "Simple, 
Medium, Advanced"
-  db_subst xserver-xfree86/config/monitor/selection-method default "Simple"
+  DEFAULT="Simple"
 fi
 
-auto_answer db_input "$(priority_ceil $PRIORITY)" 
xserver-xfree86/config/monitor/selection-method "Advanced"
+db_subst xserver-xfree86/config/monitor/selection-method default $DEFAULT
+auto_answer db_input "$(priority_ceil $PRIORITY)" 
xserver-xfree86/config/monitor/selection-method $DEFAULT
 
 db_get xserver-xfree86/config/monitor/selection-method
 case "$RET" in

Reply via email to