Closed (won't fix)
Project:
Search by Page
Version:
6.x-1.x-dev
Component:
Main Search by Page module
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
29 Nov 2011 at 16:15 UTC
Updated:
31 Jan 2012 at 21:00 UTC
In the HEAD of the search_pages page the stylesheet 'search.css' is missing.
FIX:
Include in search_by_page.module on 823:
function _search_by_page_view($envstring) {
$environment = intval(drupal_substr($envstring, 3));
++ // Add CSS
++ drupal_add_css(drupal_get_path('module', 'search') .'/search.css', 'module', 'all', FALSE);
$path = explode('/', $_GET['q'], 2);
$keys = '';
etc
Sorry for I'm not good at creating patches.
Comments
Comment #1
jhodgdonThat looks like a reasonable thing to do. I'll test and report back... Thanks! (No problem on not being able to make a patch -- for a one-line change like this it's not all that crucial.)
Comment #2
jhodgdonI'm finally getting back to this issue, sorry...
So it looks like the Search module includes this CSS file in the search_form() function, not in search_view(). So if Search by Page were to add it, it would need to go into search_by_page_form().
I'm not convinced Search by Page should include that CSS file though. At this point, adding it might screw up people's site theming... So I think I'd better close this as a "won't fix". Sorry... That search.css file only has a few lines that would even apply to Search by Page anyway.
Of course, you can just put the CSS you want into your site's theme.
Comment #2.0
jhodgdon< head > changed in HEAD