Closed vedran-kasalica closed 2 months ago
Perhaps I should motivate the PR a bit better, let me give that a shot to start off the discussion.
A logo is a simple addition to make it easier and a bit nicer to communicate about the project, both externally and in for instance the RSD and all the places where the RSD information is used (reports and such). Many of our successful projects have a logo; this one is currently a bit of an odd one out. Initially I thought it would be a bit weird to have a logo on a template, since the whole point is for the template to be instantiated without branding so that the user can apply their own. That will remain the case in the generated project. For the repo's README itself, though, and places like the RSD, it adds a little flair and professionalism.
We definitely don't want to over-engineer this, so I just took 1 minute to generate something using ChatGPT that is not too eScience branded, because we also want to welcome non-NLeSC contributors, but captures the essentials: a copier, a snake and a researcher. I'm very happy Vedran cleaned up the loose ends :) Let's not tweak it any further (at least for now).
Anybody strongly against this addition?
Nice idea adding a logo! Agree with the motivation (looks better on the RSD)
One small point: we should consider where we will be using the logo, and how it looks then. For example, the current version might not work great if you reduce it to the size it will be displayed on the RSD or as an favicon on your browser tab.
Good points! The RSD page seems to shrink the height to 144 pixels; on my screen that still works well with this logo. I don't think we need a favicon; at least I don't foresee currently us building a custom website around the template.
If nobody objects, let's go forward with the PR.
Checklist before submitting an issue
Issue details
It would be nice to make a template logo and include it in the readme. It would make the package better recognizable.