Make "Choose Color" dialog a Modal dialog

Make "Choose Color" dialog a Modal dialog

[Aditya Katti]

Hi,
Please find below a patch for Insight (srcpref.itb). Previously multiple "Choose Color" dialogs were being opened. This patch will make "Choose Color" dialog a Modal dialog.


For gdb/gdbtk/ChangeLog:
============================================gdb/gdbtk/ChangeLog===============================================

2005-08-03  Aditya Katti  <adityak@kpitcummins.com>

  * library/srcpref.itb (_pick): Makes "Choose Color" dialog a Modal dialog

==============================================================================================================

For gdb/gdbtk/library/srcpref.itb:
========================================gdb/gdbtk/library/srcpref.itb=========================================

*** insight-6.1/gdb/gdbtk/library/srcpref.itb.orig	Wed Aug  3 12:37:30 2005
--- insight-6.1/gdb/gdbtk/library/srcpref.itb	Wed Aug  3 12:37:52 2005
***************
*** 270,274 ****
  # ------------------------------------------------------------------
  itcl::body SrcPref::_pick {color win tag} {
!   set new_color [tk_chooseColor -initialcolor $color -title "Choose color"]
    if {$new_color != $color && $new_color != {}} {
      set _new(gdb/src/$tag) $new_color
--- 270,274 ----
  # ------------------------------------------------------------------
  itcl::body SrcPref::_pick {color win tag} {
!   set new_color [tk_chooseColor -initialcolor $color -title "Choose color" -parent $win]
    if {$new_color != $color && $new_color != {}} {
      set _new(gdb/src/$tag) $new_color

================================================End Of Patch==================================================


Regards,
Aditya Katti
KPIT Cummins Infosystems Limited
Pune, INDIA

RE: Make "Choose Color" dialog a Modal dialog

[Dave Korn]

----Original Message----
>From: Aditya Katti
>Sent: 05 August 2005 06:19

> Hi,
> Please find below a patch for Insight (srcpref.itb). Previously multiple
> "Choose Color" dialogs were being opened. This patch will make "Choose
> Color" dialog a Modal dialog.  

  I don't have any approval rights, but I can confirm the problem exists and
that the patch fixes it.  Thanks! :)

  [ NB. to the list: It might not seem a problem allowing multiple "Choose
color" dialogs open if they all updated their correct colours in the
palette, but in fact when several are open they fail to redraw properly, so
making it modal is a good idea. ]


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....