iwidgets for itk4

View Ticket
Login

View Ticket

Ticket Hash: 090f610de9ff1a087af4113fa563f977b71103a8
Title: tabset.itk patch to enable background/foreground colors with spaces in their names
Status: Open Type: Code_Defect
Severity: Severe Priority:
Subsystem: Resolution:
Last Modified: 2018-06-05 23:33:31
Version Found In: latest
User Comments:
bigfaceworm added on 2018-06-05 23:33:31:
If you create a tabset with the options -foreground/-selectforeground/-disabledforeground/-background/-selectbackground and specify colors using the english description like "Dark Blue"

this is due to the use of 'eval' and not properly creating the command using [list] to preserve quoting.

Here is the patch you can apply to fix the problem
-------------------------------------------------------------------
--- tabset.itk.orig	2018-06-05 16:23:54.228090301 -0700
+++ tabset.itk	2018-06-05 16:24:40.547745872 -0700
@@ -893,11 +893,11 @@
         -left             $x \
         -top              $y \
         -font             [list $itk_option(-font)] \
-        -background       $itk_option(-background) \
-        -foreground       $itk_option(-foreground) \
-        -selectforeground $itk_option(-selectforeground) \
-        -disabledforeground $itk_option(-disabledforeground) \
-        -selectbackground $itk_option(-selectbackground) \
+        -background       [list $itk_option(-background)] \
+        -foreground       [list $itk_option(-foreground)] \
+        -selectforeground [list $itk_option(-selectforeground)] \
+        -disabledforeground [list $itk_option(-disabledforeground)] \
+        -selectbackground [list $itk_option(-selectbackground)] \
         -angle            $itk_option(-angle) \
         -padx             $itk_option(-padx) \
         -pady             $itk_option(-pady) \