Fixes #1720 - ColorPicker pop up example (in ProgressBarStyles Scenario)

[tig]

Fixes

• Fixes Add ColorPicker pop-up example #1720

Pull Request checklist:

• I've named my PR in the form of "Fixes #issue. Terse description."
• My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
• My code follows the Terminal.Gui library design guidelines
• I ran dotnet test before commit
• I have made corresponding changes to the API documentation (using /// style comments)
• My changes generate no new warnings
• I have checked my code and corrected any poor grammar or misspellings
• I conducted basic QA to assure all features are working

[BDisp]

How about popup near the respective buttons on available space (below, up, right, left) as the ContextMenu does?

[tig]

How about popup near the respective buttons on available space (below, up, right, left) as the ContextMenu does?

Yea, that's a bug. I'll fix.

[tig]

WindowsTerminal_mEERHWNnTu.mp4

[BDisp]

@tig in my opinion if you click on the Foreground button the popup should appear bellow the bottom of the button. See the case of the ContextMenu:

Or above the top of the button:

[tig]

@tig in my opinion if you click on the Foreground button the popup should appear bellow the bottom of the button. See the case of the ContextMenu:

100% agree.

I am not going to spend the time now implementing this in ProgressBarStyles.cs.

Instead, it should be a built-in feature as part of

• Get rid of Toplevel - Introduce Runnable and Overlapped instead #2491

[BDisp]

100% agree.

I am not going to spend the time now implementing this in ProgressBarStyles.cs.

Instead, it should be a built-in feature as part of

Of course that this have more sense for a built-in generic popup that can be popup by any view and with any view content inside the popup.