- Issue created by @Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Moving this from project_browser queue to project queue per @leslieg.
- Status changed to Needs review
3 months ago 2:32am 13 August 2024 - Assigned to sourojeetpaul
- Status changed to Needs work
3 months ago 7:11am 13 August 2024 Hi @Chris Matthews,
The logo you provided looks pretty straightforward. I think as a logo it should represent the functionality in more visual annotations than the literal ones! Hence moving it to NW.
I'll also try to come up with my version!- Status changed to RTBC
3 months ago 1:06pm 13 August 2024 - Issue was unassigned.
- Status changed to Needs review
3 months ago 8:20am 14 August 2024 Hello there,
I've came up with my version of logo finally, can you guys kindly have a look at it?- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Thanks but the maintainer already added the other one
@VladimirAus would you please give credit to the logo creator?
- Status changed to Needs work
3 months ago 4:02pm 15 August 2024 - π¦πΊAustralia VladimirAus Brisbane, Australia
@Kristen Pol: I'm not a maintainer, so won't be able to credit.
For icon crafters: can you create merge request with logo.png added to the rrot folder of the module?
Cheers π» - Status changed to Needs review
3 months ago 6:16am 16 August 2024 Hello there,
I think there was bit of a confusion regarding the review and maintainer of the same. Also I guess @VladimirAus have somehow missed my logo, as it was provided later on :(
So @Kristen Pol can you kindly re-review the logo, as there's 2 options to choose from now!
I'm moving the issue to NR for other's to pour in their views on it...- π¦πΊAustralia VladimirAus Brisbane, Australia
@sourojeetpaul: just open new merge request with your logo.
Whever logo will be accepted by maintainers, the maintainers would just need to merge one or the other merge request.
As I'm not a maintainer of this module I can only vote ir help with MR. Hi @VladimirAus,
I don't think that'll be a good idea to do so. Opening two merge request for a single issue is not at all recommended!
Also The MR should only be raised once the logo is reviewed, approved, finalised and perfectly ready to be shipped, keeping on opening MRs for each proposed logo is not the way to do so...
Pls refer to the issue summary..Once a logo is reviewed and approved, it should be added to the root folder of your project's repo, and be named logo.png.
IMO the MR should be raised by the maintainer only in these case, as on other logo issues I've noticed the same pattern to be followed.
@Kristen Pol What's your opinion on this?- Status changed to RTBC
3 months ago 12:51pm 16 August 2024 - πΊπΈUnited States Chris Matthews
π Thanks all for chiming in on this issue. I am a maintainer for this module. @sourojeetpaul, I do sincerely appreciate the time you spent creating your version of the Project Browser logo. It's tough because the creative design process can be so incredibly subjective, which means it can be hard to choose. However, since a choice does have to be made, I'm going to run with the original logo.png β . Again, thank you for your time (giving you credit on this issue) and I hope it doesn't discourage you from creating logo designs for other projects. Cheers!
-
Chris Matthews β
committed 40cb9424 on 8.x-1.x authored by
VladimirAus β
Issue #3459441 by VladimirAus, sourojeetpaul, Chris Matthews: Project...
-
Chris Matthews β
committed 40cb9424 on 8.x-1.x authored by
VladimirAus β
- Status changed to Fixed
3 months ago 12:55pm 16 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.