style.css is not called for menu items

rkendall - May 3, 2006 - 10:58
Project:wireframe
Version:4.7.x-1.x-dev
Component:CSS
Category:support request
Priority:normal
Assigned:Unassigned
Status:closed
Description

I noticed that the css link isn't working properly on some pages, perhaps noted due to an install in a subdirectory. The homepage would work fine, but not when I navigated to other locations like admin/modules it would loose the css formatting. (note: using clean urls too)

To fix, I replaced the following line in page.tpl.php
<style type="text/css" media="all">@import "<?php print path_to_theme(); ?>/style.css";</style>
with
<?php print $styles ?>

#1

Bèr Kessels - May 3, 2006 - 11:53
Status:active» fixed

This is because BASE tag was removed. printing styles is not the solution.

Wireframe needs *only* the wireframe.css, and should not print any other styles. That is the whole purpose of it ;)

It is fixed in the latest HEAD. and in 4.7.

Bèr

#2

technigrafa - May 3, 2006 - 19:17

I am using the 4.7 version, and styles do not properly show up when clean urls are turned on.

#3

technigrafa - May 3, 2006 - 19:47
Title:css not displaying on some pages (admin)» No Header
Component:PHP» CSS

I'm not sure if the header was excluded by design, but here's how to put it in:

in page.tpl.php add:

<!-- start header -->
<div id="header">
   <?php print $header ?>
</div>
<!-- end header -->

<br class="clear" />

Right before:
<div id="leftnav">

And in style.css add:

#header {
margin: 0 1em 0 1em ;
padding: 1em;
}

#4

Bèr Kessels - May 3, 2006 - 21:13

What are you looking for exactly? It sounds like you want to use wireframe as a normal theme? Am I correct?

Wireframe is intended to work for you as a development tool. So that you can develop modules and code, in such a way that you can be sure that a theme has no effect on it!

#5

technigrafa - May 3, 2006 - 22:04

Yup. I am basing a new custom theme off of this wireframe theme. It seems to be a good clean starting point for a pure CSS theme. For this reason, it'd be cool if it worked with clean urls turned on.

But even for testing, as you say, I would think it would be nice to show the header.

#6

rkendall - May 5, 2006 - 13:34
Title:No Header» BASE issue

Cheers Bèr,
the extra / seems to do the trick, I just put the print $styles out of habit.

P.S. I agree that wireframe should focus on it's goal of being a development theme, although it is also handy as a barebones starting point for a new theme (it doesn't have all the crud most other themes have).

#7

technigrafa - May 10, 2006 - 21:38

Can you explain what you mean by adding an extra "/" ? Did that fix the short URLs issue?

#8

Bèr Kessels - May 11, 2006 - 10:50

HEADER: please open a new issue for that. I misunderstood it, and thought you wanted to include all *stylesheets* in the HTML HEADER, like the first post does by printing $styles. But "not printing the header region "is a bug.

CSS: if you do this:

<style type="text/css" media="all">@import "/<?php print path_to_theme(); ?>/style.css";</style>

Does that fix the issue?

#9

Bèr Kessels - May 21, 2006 - 09:42
Title:BASE issue» No Header

Committed

#10

rvlong - May 23, 2006 - 22:26

I'm having the same issue. Inserting the "/" did not fix the problem for me.

#11

rvlong - May 23, 2006 - 22:35
Title:No Header» style.css is not called for menu items

To clarify my previous reply, the home page is formatted correctly using style.css. However, when I select any menu item I get an unformatted page.

#12

Bèr Kessels - May 24, 2006 - 08:18
Category:bug report» support request
Status:fixed» active

Are you using the latest 4.7?

#13

rvlong - May 26, 2006 - 00:16

Yes, I'm using 4.7. I updated my php.tpl.php file to use <?php print $styles ?> which fixes most of the issues I was having. The CSS is applied to all pages now, however images like my logo only show up on my homepage but are not visible on other pages.

It's a nice clean template and I really like the fact that it's tableless and the attention to making it XHTML compliant is the main reason I'd like to go with this simple template.

I'm new to Drupal so it could be something simple that I'm overlooking. I just don't see the same problem when using other templates like grey_box or bluemarine.

Thanks for your help :)

#14

rvlong - May 26, 2006 - 20:25

I fixed this error. It was my mistake.... I was referencing the logo incorrectly in page.tpl.php. My bad.

Thanks for your time.

#15

Bèr Kessels - May 27, 2006 - 11:12
Status:active» fixed

#16

Anonymous - June 10, 2006 - 11:15
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.