The attached patch adds a checkbox to the advanced settings form that allows you to remove the themeing done in the google_cse.js file. I've got a search box that I don't want to display the 'Google CSE' background on and don't want the padding and borders that get applied, etc. The current JS overrides my background styles on the input box.

CommentFileSizeAuthor
#3 google_cse-remove-watermark-1168840-3.patch4.4 KBosopolar
google_cse_js.jpg20.26 KBtars16
remove-custom-search-background.patch3.12 KBtars16

Comments

kmonty’s picture

kmonty
He/Him
Location Berlin, DE + SF, CA
commented

I have not been able to test this patch (my project is d6). However, I'd like to voice my support in favor of this change.

malcomio’s picture

malcomio commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Related issues: +#2498613: Show watermark optionally and rewrite js code for watermark

Very similar to #2498613: Show watermark optionally and rewrite js code for watermark

osopolar’s picture

osopolar
Primary language German
Location 🇩🇪 GER 🌐
commented
Status: Active » Needs review
StatusFileSize
new google_cse-remove-watermark-1168840-3.patch4.4 KB

As promised in #2498613: Show watermark optionally and rewrite js code for watermark I have looked into the watermark issue again. I think it should be OK to completely ignore the javascript function googleCSEWatermark if the option show watermark got disabled.

I also rewrote google_cse.js so that it uses Drupal.behaviors.

retiredpro’s picture

retiredpro commented

Thanks. I worked like a charm!

  • afinnarn committed 7b4067a on 7.x-2.x authored by osopolar 
    Issue #1168840 by osopolar: Add the option to remove the styles applied...
afinnarn’s picture

afinnarn commented
Status: Needs review » Fixed

Thanks guys!

  • afinnarn committed 7b4067a on 7.x-3.x authored by osopolar 
    Issue #1168840 by osopolar: Add the option to remove the styles applied...
afinnarn’s picture

afinnarn commented
Status: Fixed » Closed (fixed)