-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Desktop: Support Windows tiles #3063
Desktop: Support Windows tiles #3063
Conversation
Useful for future tile designs, they contain suggested content guides
Images and manifest
Thanks for the pull request. How did you generate the svg files? I wonder if you could automate the process and generate this and the bitmaps using Tools/generate-images.js? In general we now use this script to automatically generate Joplin images based on what's in Assets/ImageSources. |
Actually are these svg files used for something, as I don't see them being referenced anywhere? |
No, they are not directly used anywhere. Since
I handcrafted them with Inkscape following the visual directives of VS Code and the documentation I linked. I had to tweak a little to get a balanced result.
If you mean from
Yes, I saw that, but in my opinion the Windows Tile is on the same customisation level of the MacOS Tray Icon Template ( In fact I followed the same paradigm:
|
This is a good change, and I'd like to merge it but going forward we'll generate the Joplin icons and logos dynamically and I believe this one can be too. You could add a high resolution version of your image to "ImageSources" (naming it something generic like "SquareNoBackground_150x150.png" for example) and create the two required images from it in generate-images.js The SVG files you've used as source, feel free to leave them in the Asset dir as they could indeed be useful. If you need some help getting the generate-image script to work feel free to let me know. |
Just one final confirmation before I dive into it.
Because of this, it is not as straightforward as taking the big and making the small. I could use the two SVG files to generate the two final images, which is my suggested solution. So, can you confirm which course of action I should take? |
The ideal solution if you can would be to have the J only in a png, then to compose the required images from it. So you'd create a completely transparent image, then add the J on top it at the right position, depending on the icon you want to create. The Sharp lib we use is quite powerful and would allow you to do that relatively easily. https://sharp.pixelplumbing.com/api-composite |
Yes, as a matter of fact I was already working on something similar, with a slightly different technique but that still works and is fairly scalable. As of now, I'm not totally convinced with the result. I will tweak it a little. And finally the screen where I use Windows has a resolution of 1366x768. |
No please leave them there. All the dynamically generated images are left in the repo, to make building the project from scratch easier, and because the images rarely change. |
Ok, then the PR if ready for review, as after trying different combinations of interpolation kernels and positioning tecniques, I feel this is as good as I can make it to be. |
That's great, thanks @Jackymancs4, that looks good. |
Hello,
On Windows, when Joplin is pinned on the Start Menu, it seems a little out of place, with the unisize icon in the middle of the tile.
This PR make it support the Windows Tile system, right now in it's simplest form.
File nomenclature and icon positioning has been taken from here:
https://docs.microsoft.com/en-us/windows/uwp/design/style/app-icons-and-logos
Visual Manifest from here:
https://github.com/microsoft/vscode/blob/master/resources/win32/VisualElementsManifest.xml
For documentation on the addition to the build configuration
https://github.com/electron-userland/electron-builder/issues/530
and
Application Contents - electron-builder
The color used for the accent background is the darker color found here:
https://github.com/laurent22/joplin/blob/master/Assets/JoplinIcon.svg?short_path=0d334a3
The changes only affect the Windows build artifacts.
Result