Context menu item to clear terminal

[datho7561]

Closes #3135

Signed-off-by: David Thompson davthomp@redhat.com

[codecov]

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (24e4830) 36.96% compared to head (02d6731) 36.93%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3378      +/-   ##
==========================================
- Coverage   36.96%   36.93%   -0.03%     
==========================================
  Files          77       77              
  Lines        5384     5388       +4     
  Branches     1034     1035       +1     
==========================================
  Hits         1990     1990              
- Misses       3394     3398       +4     

Files	Coverage Δ
...rc/webview/openshift-terminal/openShiftTerminal.ts	10.47% <0.00%> (-0.23%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[vrubezhny]

Perfectly works... It's just that the only one thing confuses me... See the different context menus we have (in Dark Theme):

[datho7561]

I have the "window.titleBarStyle": "custom" setting enabled. This setting makes all the context menus look like the one I added.

I think we might need to detect if this setting is enabled and switch between the two styles of context menu. The only issue is that I think the appearance varies between operating systems when "window.titleBarStyle": "custom" is disabled.

[vrubezhny]

Yes, I had "window.titleBarStyle" === "native"... With "custom" all the other menus are looking "dark" exactly as the one from the OS Terminal.

I don't know, but probably it's better if all the application menus look the same way - color scheme/fonts/etc.

[vrubezhny]

Looks good to me.

The context menu coloring issue - a color scheme on OS Terminal context menu looks different compared to other app context menus when Dark Theme and "window.titleBarStyle" === "native" preference are set - is not related to the Clear OS Terminal context menu action itself. We should probably open a different issue for this or leave as is.