There's a "Proudly built by Commerce Guys" text and graphic at the bottom of every admin screen (when the default "Shiny" theme is selected). It's hard-coded into that theme, so ordinary users can't remove it. That's not cricket: It should be implemented as a block, to give end users more control and to keep it a community project.

(Let's face it: Virtually nobody will remove it. Still, they should have the option without branching your code.)

This is related to #1789672: Remove the Kickstart logo and "Proudly built by Commerce Guys" (for the Shiny theme itself). Regardless of what happens there, the change should be made here that can't be made there (to put it into a block). A patch is posted on that issue to remove it from the theme; others will have to implement it as a block.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dudenhofer’s picture

I created a patch for shiny in preparation for this. We would need to add the 2 blocks to the installation with this HTML:

Built By block

<div id="credit" class="clearfix">
    <div class="message">Proudly built by</div>
    <div class="logo"><a href="http://commerceguys.com">Commerce Guys</a></div>
  </div>

Powered By Block

<div id="powered">
    Powered by <a href="http://drupalcommerce.org">Drupal Commerce</a>
  </div>
jsacksick’s picture

Version: 7.x-2.0 » 7.x-2.x-dev
Status: Active » Needs review
FileSize
3.11 KB

Here's the gerrit commit : https://code.drupalcommerce.org/#/c/625/

jsacksick’s picture

Updated patch with clearfix class and renamed block delta.

dudenhofer’s picture

Status: Needs review » Reviewed & tested by the community

I tested this patch, it works just fine

jsacksick’s picture

Status: Reviewed & tested by the community » Fixed

Thanks! Merged

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.