Skip to content

Replace view icon for terminal view#2067

Merged
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:terminal-new-view-icons
Jul 21, 2025
Merged

Replace view icon for terminal view#2067
HeikoKlare merged 1 commit intoeclipse-platform:masterfrom
HeikoKlare:terminal-new-view-icons

Conversation

@HeikoKlare
Copy link
Copy Markdown
Contributor

This is a proposal to exchange the terminal view icons with a new version. The existing icons are only present as raster graphics and are based on elements that do nowadays not fit that well anymore. At the same time, other icons for terminals became common in other applications. The proposed icons (for the view itself and the toolbar item to create a new view) align with such common terminal icons.

These are the existing and the proposed icons for comparison:
Existing:
terminal_view
new_terminal_view

Proposal:
terminal_view
new_terminal_view

Screenshots

Taken on Windows at different zooms.

100%
image

150%
image

175%
image

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 20, 2025

Test Results

 1 947 files  ±0   1 947 suites  ±0   1h 40m 49s ⏱️ + 4m 0s
 4 720 tests ±0   4 696 ✅ ±0   24 💤 ±0  0 ❌ ±0 
14 160 runs  ±0  13 993 ✅ ±0  167 💤 ±0  0 ❌ ±0 

Results for commit c7c0245. ± Comparison against base commit e3841c0.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare marked this pull request as ready for review July 20, 2025 13:56
@BeckerWdf
Copy link
Copy Markdown
Member

My only question:
Do we really want to have these tiny stars:

image

In the "File > New" main menu icons we also don't have them.

@HeikoKlare
Copy link
Copy Markdown
Contributor Author

HeikoKlare commented Jul 21, 2025

My only question:
Do we really want to have these tiny stars:

I think we should remove them. I just took the "add overlay" from the icon for creating a new project, which contains those stars. I didn't check other icons with an "add overlay", which all do not contain such stars. And I would remove the tail underneath the stars as well, so only keeping the "+" symbol.

@HeikoKlare HeikoKlare force-pushed the terminal-new-view-icons branch from 35fdde4 to 67d52fa Compare July 21, 2025 08:05
@BeckerWdf
Copy link
Copy Markdown
Member

I just took the "add overlay" from the icon for creating a new project, which contains those stars.

If you tell me which file this is I can have a look at it and provide a PR that removes them also there.

@HeikoKlare
Copy link
Copy Markdown
Contributor Author

If you tell me which file this is I can have a look at it and provide a PR that removes them also there.

It only seems to be present in the rather generic "new" icon, but that one is spread across multiple bundles. Just a few I found via the image browser:

image image image image

I first thought it was the general "new project" icon, but then I found that the "new project" icon is inconsistent in the "New" toolbar drop-down menu and in the "New" wizard that is opened by that tool item:
image
image

Not sure if that has always been like that or if it has been mixed up when replacing PNGs with SVGs.

This exchanges the terminal view icons with a new version. The existing
icons are only present as raster graphics and are based on elements that
do nowadays not fit that well anymore. At the same time, other icons for
terminals became common in other applications. The proposed icons (for
the view itself and the toolbar item to create a new view) align with
such common terminal icons.
@HeikoKlare HeikoKlare force-pushed the terminal-new-view-icons branch from 67d52fa to c7c0245 Compare July 21, 2025 17:16
@HeikoKlare HeikoKlare merged commit e58293f into eclipse-platform:master Jul 21, 2025
18 checks passed
@HeikoKlare HeikoKlare deleted the terminal-new-view-icons branch July 21, 2025 19:07
@BeckerWdf
Copy link
Copy Markdown
Member

I first thought it was the general "new project" icon, but then I found that the "new project" icon is inconsistent in the "New" toolbar drop-down menu and in the "New" wizard that is opened by that tool item

So the "New > Project", "New >Example" and the "New > Other" all use this icon with the stars and the wizard that opens also has these stars in the wizard banner. So I would say this is consistent.
Or what do you exactly mean with inconsistent?

We anyway can decide that the stars in both icons (toolbar and wizard banner) should be removed from both. But this is more a question of "style" and "taste" and not one of consistency.

Not sure if that has always been like that or if it has been mixed up when replacing PNGs with SVGs.
I don't think this got mixed up.

@HeikoKlare
Copy link
Copy Markdown
Contributor Author

So the "New > Project", "New >Example" and the "New > Other" all use this icon with the stars and the wizard that opens also has these stars in the wizard banner. So I would say this is consistent.
Or what do you exactly mean with inconsistent?

I referred to the Project entry in the "New" drop-down menu and the Project entry inside the New wizard having different icons:
image
image
But I just noticed that the formed does not start the wizard to create a new plain project but starts the wizard for different kinds of projects (as indicated by the ... in the drop-down menu entry). I have to admit that I, as a user, am not able to easily distinguish all these different usages of "Project" and have to try on my own, but that's nothing to blame the icons for and also nothing I consider anything severe and necessary to address.

We anyway can decide that the stars in both icons (toolbar and wizard banner) should be removed from both. But this is more a question of "style" and "taste" and not one of consistency.

Yes, we could do that, but it's also just fine for me as is. This all boils down to that I just made a bad choice for the icon to take the "add" symbol from and that everything can be considered just fine as is. So from my side, there is no need to change anything and we could close the discussion.

@BeckerWdf
Copy link
Copy Markdown
Member

Yes, we could do that, but it's also just fine for me as is. This all boils down to that I just made a bad choice for the icon to take the "add" symbol from and that everything can be considered just fine as is. So from my side, there is no need to change anything and we could close the discussion.

Same for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants