Ethereal-cvs: [Ethereal-cvs] cvs commit: ethereal/gtk capture_dlg.c color_dlg.c dfilter_expr_d
guy 2004/02/12 16:24:29 CST
Modified files:
gtk capture_dlg.c color_dlg.c
dfilter_expr_dlg.c file_dlg.c gui_prefs.c
main.c menu.c rtp_analysis.c rtp_stream.c
simple_dialog.c
Log:
The HIG's and/or GUI toolkit documentation for:
Mac OS X
GNOME
Qt
KDE
Windows
all indicate that {message,alert} boxes are modal, at least for the
window to which they apply. (Presumably the idea is that not forcing
the user to pay attention to the alert box, and allowing more than one
alert box to be up for a given window, causes more problems than not
letting the user do stuff to that window in order to figure out what the
underlying problem is or figure out what to do to fix it - the message
should be sufficient, in most if not all cases, to let you know what the
problem is.)
Make "simple_dialog()" unconditionally make the alert box modal, and get
rid of ESD_TYPE_MODAL. XXX - we need to make it possible to make an
alert box modal for a given window, rather than just the top-level
window.
Revision Changes Path
1.104 +2 -3 ethereal/gtk/capture_dlg.c
1.42 +2 -3 ethereal/gtk/color_dlg.c
1.51 +5 -6 ethereal/gtk/dfilter_expr_dlg.c
1.96 +3 -5 ethereal/gtk/file_dlg.c
1.64 +4 -4 ethereal/gtk/gui_prefs.c
1.395 +4 -6 ethereal/gtk/main.c
1.163 +2 -3 ethereal/gtk/menu.c
1.36 +24 -25 ethereal/gtk/rtp_analysis.c
1.14 +3 -3 ethereal/gtk/rtp_stream.c
1.27 +3 -4 ethereal/gtk/simple_dialog.c