- Issue created by @lostcarpark
- 🇮🇪Ireland lostcarpark
I have uploaded PNG and SVG versions of the logo, both sized to 512x512 pixels.
Here's a preview of the PNG. I had to give the SVG a .TXT extension, so I don't think I can embed here. However they should both look the same unless you zoom in.
In terms of file size, the PNG is 9KB, while the SVG is 5KB. Both are under the Project Browser requirement of 10KB.
I would favour using the SVG since it's smaller, and we already use a SVG for the module default image.
- 🇮🇪Ireland lostcarpark
I've done some optimisation of the SVG file, and got it down to 2.5KB.
- Status changed to Needs review
4 months ago 5:00pm 23 April 2025 - 🇮🇳India BhumikaVarshney Delhi
Hi
I took pull from the MR and everything is working fine
Now instead of using png you have replaced it with svg if sometime we zoom in zoom out the svg is looking perfect.
I have added screenshot of the zoomin svg - 🇺🇸United States chrisfromredfin Portland, Maine
I believe the logo is being set by the recipe backend to be the png we don't like. So while this introduces a fallback, will the fallback be used for recipes?
https://git.drupalcode.org/project/project_browser/-/blob/2.0.x/src/Plug...
Do we need to change it there?
- First commit to issue fork.
- 🇮🇳India JatinGupta40
Resolved the conflicts.
@chrisfromredfin I believe yes we need to change it from here -
https://git.drupalcode.org/project/project_browser/-/blob/2.0.x/src/Plug... - Status changed to Fixed
21 days ago 5:16pm 30 July 2025 - 🇺🇸United States chrisfromredfin Portland, Maine
OK, I was mistaken about the fallback anyway - cuz then it shows the puzzle piece and not the recipes logo. I had to add "base:" to the URI to get it to work, so we're good here. Merging to 2.0.x and forward-porting to 2.1.x
-
chrisfromredfin →
committed 28b84bea on 2.0.x authored by
lostcarpark →
Issue #3494655 by chrisfromredfin, lostcarpark, jatingupta40,...
-
chrisfromredfin →
committed 28b84bea on 2.0.x authored by
lostcarpark →
-
chrisfromredfin →
committed a5a302c7 on 2.1.x
Issue #3494655 by chrisfromredfin, lostcarpark, jatingupta40,...
-
chrisfromredfin →
committed a5a302c7 on 2.1.x
Automatically closed - issue fixed for 2 weeks with no activity.