From: <no...@tc...> - 2025-08-05 13:33:16
|
Automated mail by fx, on behalf of no...@tc... Ticket Change [c03b15547c1c9ec011a7b5fbf4a8dd868edc9fea4930e5fefbfc0352f74f0d4b] [Color picker: 11 user colors, why not 12 ?] By anonymous For BWidget On 2025-08-05T13:31:30.413 Details https://core.tcl-lang.org/bwidget/tinfo?name=c03b15547c1c9ec011a7b5fbf4a8dd868edc9fea4930e5fefbfc0352f74f0d4b Ticket https://core.tcl-lang.org/bwidget/tktview/3f82f0429d5e18adc1a52b5e898ca905a6804b2b Changed Fields icomment: Hello Harald, in Addition to my last comment: The 12. color was not defined because the place for it is used in the popup menu for the palette symbol/open dialog-button. But in dialog mode we can use the proposed 12. defined color (or more lines too). Therefore it has to be defined, as proposed. To get *NO* new line in popup-mode, a second change is needed, just after combine the base- and user-colors together -> line 110: set colors [concat $_baseColors $_userColors] ++++ # remove last user color to get space for palette button to open dialog set colors [lreplace $colors end end] ==== Then the rest goes right from -> line 111: foreach color $colors { This solution is fine for more lines of user colors too. That's all! Please do the change and commit it. Thank You and best greetings Thomas W. login: anonymous username: TWu ------------------------------------------------------------ See Tcl/Tk development @ http://core.tcl-lang.org/ ------------------------------------------------------------ |