Bwidget Source Code
Ticket Change Details
Not logged in
Overview

Artifact ID: c03b15547c1c9ec011a7b5fbf4a8dd868edc9fea4930e5fefbfc0352f74f0d4b (Awaiting Moderator Approval)
Ticket: 3f82f0429d5e18adc1a52b5e898ca905a6804b2b
Color picker: 11 user colors, why not 12 ?
User & Date: anonymous 2025-08-05 13:31:30
Changes

  1. 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.
    
  2. login: "anonymous"
  3. mimetype: "text/plain"
  4. username: "TWu"