treemacs icon indicating copy to clipboard operation
treemacs copied to clipboard

Very blurry image icons in Retina screen

Open memeplex opened this issue 4 years ago • 12 comments
trafficstars

See for example:

image

Presumably they are rendered as for a 96 dpi screen and then scaled up x 2.

I'm using emacs-28 branch built from source with svg support. (frame-scale-factor) is correctly reported as 2. My laptop is a MacBook Pro 2017.

Also reported to lsp-mode in https://github.com/emacs-lsp/lsp-mode/issues/3143

memeplex avatar Oct 09 '21 19:10 memeplex

Most of treemacs' icons are 22x22 px sized pngs, so the blurriness is to be expected with such a scale factor.

Converting them to svgs has been on my nice-to-have list for a while now, but it'll take me a while.

Alexander-Miller avatar Oct 10 '21 10:10 Alexander-Miller

Converting them to svgs has been on my nice-to-have list for a while now, but it'll take me a while.

What about using these ones https://github.com/microsoft/vscode-icons ?

Here is another bunch https://github.com/vscode-icons/vscode-icons.

Also it would be great if you worked together with the lsp guys (https://github.com/emacs-lsp/lsp-mode/issues/3143) and maybe company/company-box also. There are a number of problems making icons a PITA in emacs, including coordination issues.

  • font icons have alignment issues, it seems impossible to get them 100% right.
  • there is no svg set but emacs has already moved to compiling with svg support by default.
  • inconsistent icons across projects (company, lsp, treemacs).

So you may agree on providing at least one svg icon set that is consistent across the board. I don't know if there are licensing issues with the vscode set but: it's pretty, it's svg and it's "coordinated" by default.

memeplex avatar Oct 10 '21 17:10 memeplex

Thinking about it, I believe it would be better if this icon set were distributed as part of all-the-icons or other new "icon distro package" instead of duplicating and fragmenting it across many projects. Most icons have a clear meaning which will likely be enough to achieve consistency across projects without explicit coordination. This is similar to the situation with font icons right now except that they bring a host of alignment problems and that their semantics is not always clear (being a mishmash of icons not necessarily designed for programming environments) which doesn't favor consistency.

memeplex avatar Oct 10 '21 18:10 memeplex

What about using these ones https://github.com/microsoft/vscode-icons

That's my plan. But that's still replacing about 200 individual icons, so it'll take me a while to catch em all.

Also it would be great if you worked together with the lsp guys

Already happening - just look at lsp-treemacs.

font icons have alignment issues

That's because font icons are annoying buggers that aren't monospaced. So your best bet is add a tab to them and hope that it'll mitigate the ugliness. Sometimes it works, and sometimes dired-mode has a default tab-width of 8.

Thinking about it, I believe it would be better if this icon set were distributed as part of all-the-icons or other new "icon distro package"

I had the same thought, but it'll have to wait. This nifty stale bot I have just installed reminded me of a bunch of old issues, some of which are up to two years old now. I want to work my way through the old stuff before I start a major undertaking like this.

Alexander-Miller avatar Oct 11 '21 05:10 Alexander-Miller

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Dec 10 '21 05:12 stale[bot]

Stayin alive!

Alexander-Miller avatar Dec 13 '21 18:12 Alexander-Miller

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Feb 11 '22 18:02 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity.

stale[bot] avatar Jun 28 '22 17:06 stale[bot]

Stay alive!

cfclrk avatar Oct 23 '22 18:10 cfclrk

Stay alive!

Yes, but also fyi making stay-alive comments is no longer strictly necessary now. I've turned off the part of the bot where it closes tickets, it only acts as a reminder now.

Alexander-Miller avatar Oct 24 '22 15:10 Alexander-Miller

This issue has been automatically marked as stale because it has not had recent activity.

stale[bot] avatar Dec 23 '22 17:12 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity.

stale[bot] avatar Feb 22 '23 16:02 stale[bot]