- Issue created by @cedewey
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Updating this issue based on the new template from @leslieg.
- Assigned to Tirupati_Singh
- Issue was unassigned.
- Status changed to Needs review
3 months ago 6:48am 29 August 2024 - 🇮🇳India Tirupati_Singh
Hi @kristen pol, I've created a logo for the module. Please have a look.
- 🇧🇪Belgium joevagyok
Thanks @tirupati_singh, small note from my side, if I may.
It looks good in general and the style looks similar to the examples mentioned. However, not sure how big the project browser will show these images, but if it will be small, then the numbers and the "Maxlength" text would be too small I think. But I would refer to @kristen pol in this matter.
If we wish to have the name of the module in the image, please respect the way it is on drupal.org page, "MaxLength" in this case.
Otherwise, the HTML attribute could be used, such as "#maxlength" which is self-explanatory as well. Hi @tirupati_singh,
I somewhat agree with @joevagyok, The PB logo will be shown in a much smaller dimension, hence it'll be better if we can incorporate the suggested changes.
Though I won't prefer using the HTML attribute on the logo, rather respecting the way it has been shown on the project page can be a good alternative!
Hence moving it to NW again :)- 🇮🇳India Tirupati_Singh
Hi all! I've updated the logo as per the suggested changes. I've increased the logo text and updated the module name text case as per the module page. Please review the changes.
Thanks!
Hi @tirupati_singh,
Thanks for reworking on it. I can see the feedbacks have been resolved properly now, and also limiting to the PB logo constraints. Hence I think we can move the issue forward to RTBC++- 🇨🇿Czech Republic Petr Illek
Hi,
I accidentally stumbled on this issue and got some notes and ideas.I think the logo is too complicated.
- It does not need to contain the module name, as in most of the cases the logo will be accompanied by the module name in a text variant.
- The most important part (the counter) is the smallest element on the image
- The count is too specific (does the module allow another values or not?)
- The count is wrong – the text in the field "MaxLength" is 9 characters :D
I did try to simplify the logo to the very base of the module feature.