The style worked on Linux, and Mac, and then on Windows, some custom CSS was being applied. Do we really need to be doing that?

CommentFileSizeAuthor
#2 normalbox.png20.01 KBrobloach
messwithwindows.patch808 bytesrobloach

Comments

mfb’s picture

mfb
they or he
commented

I borrowed this particular logic from elsewhere so I don't know why it was needed. I can't test this right now as I don't have windows (and I don't have time to fire up VirtualBox). So it would help if you could answer your own question :)

robloach’s picture

robloach
he/him
commented
StatusFileSize
new normalbox.png20.01 KB

Here's a screenshot of it on Windows and Linux with and without the patch, under Garland:

normalbox.png

... It shouldn't be the module's job to apply the styling. Theme's should be doing that ;-) . Either that, or move those CSS styles into google_cse.css so they're applied under all platforms.

malcomio’s picture

malcomio commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

Tidying up the issue queue, and marking issues as won't fix where the version is no longer supported.