Icy Phoenix

This perhaps isn't a bug, so much, as a potential source of annoyance / confusion, and I think this goes back a couple of versions of IP, but I couldn't see mention of it on this site (except where I recently mentioned it Here.

Basically the column widths are completely overridden in the template files, e.g.:
templates/mg_themes/layout3_column.tpl

Left row is created under <td width="180" valign="top">

Right row is also created under <td width="180" valign="top">

So no matter what people specify in the cms config page, it will be ignored.

I think this is not a bug.

You are pointing to standard CMS template not to the templates being used for global blocks.

Columns for global blocks are inserted in portal_page_headerleft.tpl and portal_page_tailright.tpl which should have the correct vars: {HEADER_WIDTH} and {FOOTER_WIDTH} respectively.

Let me know.

No it doesn't work I'm afraid... I did see the variables in portal_page_headerleft.tpl and right, but as nothing happened when I played with the cms configuration, i assumed the layout page might have something to do with it (but as you say, that wouldn' effect site wide, but yes, it is the same width 180 on every page, whatever I set in CMS...) Thinking about it now, I'm wondering if perhaps when you changed $portal_config to $cms_config_vars, maybe this bit of the change is left out?

For what I can see the var is assigned and parsed correctly.

But I will do some extra check.

Vars are assigned in page_header.php and page_tail.php respectively:



If you try to insert a DIE just after those declarations you will see that the var is correctly passed:


Are you sure you don't have something in the column which doesn't allow the column to be sized as you wish? (e.g.: text too wide...)

Nope not that... you've got it right in page_header.php:
$template->assign_var('HEADER_WIDTH', $cms_config_vars['header_width']);

DB is also definitely being updated, so not sure why that portal header td is not being set... :?
Best thing would be to echo the val I guess ;)

BTW as said at the start, not so much a bug, but possible source of annoyance - because the main portal page layout is still fixed - why would somebody want a different width on every other global blocked page but the homepage default 3 / 2 column layout? Or was you thinking of changing the way the standard templates work so they also integrate with global blocks?

Ha, I think we try and post at the same time! ;)

moreteavicar wrote: [View Post]

BTW as said at the start, not so much a bug, but possible source of annoyance - because the main portal page layout is still fixed - why would somebody want a different width on every other global blocked page but the homepage default 3 / 2 column layout? Or was you thinking of changing the way the standard templates work so they also integrate with global blocks?

You're right but this is another point...

We should add options in the layout EDIT/ADD feature in CMS.

It should not be tough, I may have a play with it one day. :wink:

:oops: I discovered why it isn't changing on the other pages... I forgot I defined another class for the left nav links in the stylesheet, as I was going to play with it... in there I fixed the width of the navbar... it is so long ago when I did it... that I totally forgot about it... sorry! (if you're wondering why fix it there, I had to fix the width in order to have cell images that changed with mouse-over, it had to be display:block... bit like what I did on one of my old stand-by sites: http://icps2007.awardspace.info/ICPS/ )

moreteavicar wrote: [View Post]

:oops: I discovered why it isn't changing on the other pages... I forgot I defined another class for the left nav links in the stylesheet, as I was going to play with it... in there I fixed the width of the navbar... it is so long ago when I did it... that I totally forgot about it... sorry! (if you're wondering why fix it there, I had to fix the width in order to have cell images that changed with mouse-over, it had to be display:block... bit like what I did on one of my old stand-by sites: http://icps2007.awardspace.info/ICPS/ )

No problem for that.

What I do not understand is why you need the diplay property to be block... couldn't just use the CSS properties to change background? There are a lot of free script which demonstrate how to do that... If I didn't misunderstand what you need... :oops:

Ah well, I am using css properties to do exactly that, but if you want to do it with a cell image, and not just define a color, then can't just tile a link with an image, you need to have a fixed image width and use display:block - at least I found this to be the case a couple of years ago. I vaguely recall seeing some demonstration's by other people that did the same, and the links are then in their own td cells (which adds come complications to overall header for e.g., you need to put them in their own table) anyway thats all water under the bridge, maybe there is a better way, but I'm not really keen on using cell images anymore... I'm more into the less is more look ;)