Hi,

I'd like to know how to change the logo used for the "Share" button in the addthis block. Preferably WITHOUT hacking into the module as this will break upgrade path(s) (Similar to the options provided in version 7.x-2.x of this module...)

Thanks

CommentFileSizeAuthor
#4 ability-to-change-main-button-image-1515424.patch2.27 KBkapil.ropalekar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lee20’s picture

lee20 CreditAttribution: lee20 commented

Subscribing. This feature is very handy and if it exists in 4.x please point out how it can be done.

lee20’s picture

lee20 CreditAttribution: lee20 commented
Title: Change main "button" » Ability to change main "button" image
Category: support » feature

I was able to override theme_addthis_element() in order to override the button image. But it would be ideal if this could be done without having to use a custom hook.

lee20’s picture

lee20 CreditAttribution: lee20 commented
Issue summary: View changes

Change

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle at Hannemyr Nye Medier AS commented
Priority: Major » Minor
Issue summary: View changes

Downgrading this to "minor", since it is a cosmetic issue (please see guide to Priority settings).

This will not be added to the project until somebody gets a patch through to RTBC.

kapil.ropalekar’s picture

kapil.ropalekar CreditAttribution: kapil.ropalekar at Cybage Software Pvt Ltd. for Cybage Software Pvt Ltd. commented
FileSize
ability-to-change-main-button-image-1515424.patch2.27 KB

I have created a patch file for this issue. Can you kindly review the same.

Ability to add configurable button settings has been added as a fieldset under the Basic configuration page settings.
Reference path : admin/config/user-interface/addthis

Fieldset : ADDTHIS BUTTON SETTINGS

kapil.ropalekar’s picture

kapil.ropalekar CreditAttribution: kapil.ropalekar at Cybage Software Pvt Ltd. for Cybage Software Pvt Ltd. commented
Assigned: Unassigned » kapil.ropalekar
Status: Active » Needs review

Assigning issue to self and changing status to Needs review.

ashwinsh’s picture

ashwinsh
He/Him
Primary language English
Location Pune
CreditAttribution: ashwinsh as a volunteer and commented

#4 works for me, thank you kapil.ropalekar

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle at Hannemyr Nye Medier AS commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch, kapil.ropalekar!

This has now been committed to the latest snapshot of the 7.x-4.x branch.

kapil.ropalekar’s picture

kapil.ropalekar CreditAttribution: kapil.ropalekar at Cybage Software Pvt Ltd. for Cybage Software Pvt Ltd. commented
Status: Reviewed & tested by the community » Fixed

Thanks gisle for committing the patch to the latest snapshot of the 7.x-4.x branch.

Marking this as fixed.

Status: Fixed » Closed (fixed)

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