Skip to content

fix WidgetColor import Issue #3776#3777

Draft
shroffk wants to merge 2 commits intomasterfrom
issue_3776
Draft

fix WidgetColor import Issue #3776#3777
shroffk wants to merge 2 commits intomasterfrom
issue_3776

Conversation

@shroffk
Copy link
Copy Markdown
Member

@shroffk shroffk commented Apr 14, 2026

fixes #3776
maintains backward compatibility as described in the comments

Checklist

  • Testing:

    • The feature has automated tests
    • Tests were run
    • If not, explain how you tested your changes
  • Documentation:

    • The feature is documented

@shroffk shroffk requested a review from kasemir April 14, 2026 20:11
@kasemir
Copy link
Copy Markdown
Collaborator

kasemir commented Apr 14, 2026

Not sure I want to update every script on our control system.
Maybe add the old WidgetColor back in:

package org.csstudio.display.builder.model.properties

// Class in original package that wraps the new one to satisfy
// existing scripts
public class WidgetColor extends org.phoebus.ui.color.WidgetColor
{

}

@sonarqubecloud
Copy link
Copy Markdown

@shroffk shroffk added the wip work in progress label Apr 14, 2026
@shroffk shroffk marked this pull request as draft April 14, 2026 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wip work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scripts that use WidgetColor are broken

2 participants