- Issue created by @Kristen Pol
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Move to project's queue per @leslieg.
- Assigned to sourojeetpaul
- Issue was unassigned.
- Status changed to Needs review
9 months ago 6:59am 19 August 2024 It was a bit tricky to showcase the functionality visually. This is what I came up with after a lot of brainstorming. Pls have a look...
- Status changed to Needs work
9 months ago 4:30pm 19 August 2024 - 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Thanks! I don’t think it needs the person in it and it would be good if it looked more like a form
- Status changed to Needs review
9 months ago 5:43pm 19 August 2024 Just one question Kristen,
If we remove the person from the logo then how we're going to decipher the 'clienside' aspect of the validation?
Any thoughts on that?!- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
I see. I guess it’s fine to keep it. Just looks more cluttered IMO. We don’t have a “server side validation” module so I’m not sure it’s really necessary though. Perhaps have both versions and the maintainer can decide?
Ok @kristen pol,
I'm sharing 3 variants for the maintainer to choose from...- Status changed to RTBC
8 months ago 8:03am 19 September 2024 - 🇮🇳India manojsaha Kolkata
Hello @sourojeetpaul
I like the first version among all the logos because it follows the proper guidelines for the requirements of Drupal module logo design. On the other hand, anyone can clearly understand that the form is for the client's side. So, I think this is one can be a good option.
I am moving the issue to RTBC++. - First commit to issue fork.
- Merge request !27Issue #3459445: Project Browser: Create a logo for Clientside Validation → (Merged) created by nikunjkotecha
- 🇮🇳India nikunjkotecha India, Gujarat, Rajkot
Thanks everyone for the great work, can someone please help review and approve the MR https://git.drupalcode.org/project/clientside_validation/-/merge_request... ?
I have lost touch with Drupal projects so not really sure MR is exactly as per requirement or we need something else?
Hi @nikunjkotecha,
Thanks for taking your time out and having a look at this! AFAIK for PB logo issues an MR with the logo.png is enough! I believe we're good to go now :)-
nikunjkotecha →
committed e0e54be1 on 4.0.x
Issue #3459445 by nikunjkotecha, sourojeetpaul, kristen pol, manojsaha:...
-
nikunjkotecha →
committed e0e54be1 on 4.0.x
- 🇮🇳India nikunjkotecha India, Gujarat, Rajkot
Thanks, MR merged, will release new version soon.
Automatically closed - issue fixed for 2 weeks with no activity.