- Issue created by @lostcarpark
- 🇮🇳India anweshasinha
Hi,
I created a logo for drupal commerce which includes Centarro logo. Please review it. - 🇺🇸United States rszrama
Thanks for trying, but we won't be using that.
- 🇮🇳India anweshasinha
Hi,
I have created the logo again. Can you please review it. - 🇮🇪Ireland lostcarpark
Hi @anweshasinha,
I believe, Centarro, who maintain the Commerce project, intend to design a Commerce Core logo in house.
- First commit to issue fork.
- Merge request !200Revert "Issue #3348311 by rhovland: Add Payment link on order view page." → (Open) created by BramDriesen
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7last update
about 1 year ago Not currently mergeable. - last update
about 1 year ago 786 pass - Status changed to Needs review
about 1 year ago 7:07pm 4 December 2023 - 🇧🇪Belgium BramDriesen Belgium 🇧🇪
In an attempt to move this forward, adding the logo (in line with all other commerce logo's proposed) and based on this comment:
For what it's worth, we're using the Centarro branding generally for Drupal Commerce work, both for the sake of internal (Drupal community) and external (broarder eCommerce marketplace) communication.
- 🇧🇪Belgium BramDriesen Belgium 🇧🇪
Note that the logo should be committed to the default branch as well (doesn't seem to be 3.0.x yet)
- 🇺🇸United States Kristen Pol Santa Cruz, CA, USA
Hiding the old logos to avoid confusion.
- Status changed to Closed: won't fix
3 months ago 3:17pm 18 September 2024 - 🇺🇸United States leslieg
Closing this issue. The maintainer weighed in quite a while ago on the logo issue.
- Status changed to Needs review
3 months ago 6:19pm 18 September 2024 - 🇺🇸United States leslieg
Sorry @ryan, I thought this was in the Project Browser queue which I am trying to clear out. I put Status back as Needs review.
- 🇳🇴Norway zaporylie
zaporylie → changed the visibility of the branch 3372681-commerce-core-logo to hidden.
- Merge request !387Issue #3372681 by zaporylie: Commerce Core Logo for Project Browser → (Merged) created by Unnamed author
- 🇳🇴Norway zaporylie
Added the logo with a purple background in 512x512 resolution. My understanding is that it is in line with the requirements for Project Browser but I couldn't find a canonical resource to back up my understanding.
- 🇳🇴Norway zaporylie
One more note - we will soon be tagging a new release of Drupal Commerce and while it is not critical to get this in ASAP it would be nice to use that opportunity to have the logo featured in a tagged release.
- 🇺🇸United States leslieg
The logo meets the requirements of 512x512, file size is close to 10k and the logo is correctly named logo.png. The file is in the root folder of the repository. (note - this need to be in the default branch). Also the existing logo needs to be removed as the first image on the d.o project. Please check that @zaporylie
- 🇺🇸United States rszrama
Looks good to me, and we can certainly update the project page images. What's best practice with respect to Project Browser? Should we be showing administrative interfaces or perhaps other visuals? Happy to look at examples that look great in context and do likewise.
-
jsacksick →
committed 75c85f3f on 3.0.x authored by
zaporylie →
Issue #3372681 by zaporylie: Commerce Core Logo for Project Browser
-
jsacksick →
committed 75c85f3f on 3.0.x authored by
zaporylie →