theia icon indicating copy to clipboard operation
theia copied to clipboard

Codicon color and URI support to TerminalOptions

Open dhuebner opened this issue 1 year ago • 4 comments

This is a copy of the original PR #12861 First commit was cherry picked from https://github.com/FernandoAscencio/theia/commit/ba3dda24e7fac3ac81522d61d2ad531b4a94f8c9 In the second commit, I applied the suggested changed from the original PR.

The original description below

What it does

Closes https://github.com/eclipse-theia/theia/issues/12074

This commit adds URI and { light, dark } support for terminal icons. This commit adds ColorTheme support for ThemeIcons Support for ExtensionTerminalOptions#iconPath

How to test

Open Theia Install from VSIX the test extension Check the result follow VSCode test results seen below:

Review checklist

Reminder for reviewers

dhuebner avatar Feb 22 '24 15:02 dhuebner

@tsmaeder Thanks for your re-view, I will have a look today.

dhuebner avatar Mar 22 '24 09:03 dhuebner

@dhuebner the license check fails. Do we need an ip ticket here?

tsmaeder avatar Mar 22 '24 14:03 tsmaeder

@tsmaeder It hard to say for me what exactly fails from reading the error log. The original PR, however, didn't fail https://github.com/eclipse-theia/theia/pull/12861 And I added no additional dependencies.

dhuebner avatar Mar 26 '24 08:03 dhuebner

@dhuebner, there was a message yesterday from Thomas on Theia dev list about those IP checks failing. You may rebase on top of master, that should fix the issue. https://github.com/eclipse-theia/theia/pull/13524 is the PR fixing the issue.

rschnekenbu avatar Mar 26 '24 08:03 rschnekenbu

@dhuebner @tsmaeder I've updated the PR myself. I've added support for custom icons for quick pick items and quick pick buttons. You can find an example plugin for that here. Simply use the Quick Input Samples command. Use the multiStepInput option to test that the button icons work as expected. With that I was able to align the icon code of terminals and quick picks.

msujew avatar May 27 '24 17:05 msujew

@msujew After properly rebuilding the local state it works for me!

Bildschirmfoto 2024-06-14 um 13 52 27

I noticed that after reloading the host, custom terminal (restored) icons are not rendered, see below. ~~Also the default terminal Terminal 5 doesn't have an Icon, should we use the default icon here?~~ I've added a default icon for the shell terminals.

Bildschirmfoto 2024-06-14 um 13 50 49

dhuebner avatar Jun 14 '24 11:06 dhuebner

@JonasHelming I (re)wrote most of this PR. I'm not in a good position to review it. I already pinged @tsmaeder for a review.

msujew avatar Jun 20 '24 10:06 msujew

It's on my list of todo's.

tsmaeder avatar Jun 20 '24 11:06 tsmaeder

@msujew how do I test this? Where do the icons show up?

tsmaeder avatar Jun 20 '24 14:06 tsmaeder

@tsmaeder

For the quick pick icons:

You can find an example plugin for that here. Simply use the Quick Input Samples command. Use the multiStepInput option to test that the button icons work as expected.

For the terminal icons: You need to install the linked plugin and run the associated commands. For example Green File Codicon Icon Terminal (they all end on Terminal).

msujew avatar Jun 20 '24 16:06 msujew

@msujew What is the state of this, ready for a re-review?

JonasHelming avatar Jun 26 '24 08:06 JonasHelming