Remove bottom margin in slider

[Info 3bits]

Remove 20px bottom margin in slider to join the regions

[Jeremy Hopkins]

I think the reason for this margin was to put a gap between slider and the info box when info box is in use. Screen shot attached. I am not sure if this is what you were referring to?

[Info 3bits]

Yes, that's right.

The idea is to remove the margins in the backgrounds but not in the content (except the slider)

Take a look to the test site: http://testlab.ws/bcu/

This design is the most common actually in other templates: WP, Joomla, .. Just take a look to: http://demo.rockettheme.com/joomla-templates

[Jeremy Hopkins]

Ok I see why you want to do that but making that a default would cut across the way the theme is being used elsewhere. Can this be added as an option / admin setting default to "margin on" and have the option to turn "margin off"?

[Info 3bits]

Maybe it could be displayed to full width in full screen mode ?

[Jeremy Hopkins]

Well I think the use of full width in both modes is desirable and something I wanted to add at some point anyway so that we could bring our Moodle more into line with our website which has full width slider and fixed width content:

http://www.coventry.ac.uk/

So if this were added as an option we could do something like that, which is not disimilar from things you have done with the theme already.

[Jeremy Hopkins]

... looking at the alterations you made for this I dont think it would be hard to add an option, I think you just removed / took it out of "container"?

[Info 3bits]

Yes, the full width depends of the use of the container class.

IMHO, the full width looks much better. And the idea is exactly the same you applied to the corporate Coventry corporate site. But could be a setting that applies or not the class to the regions.

[Jeremy Hopkins]

I think it works better too, but this theme is in use on perhaps 1000 live sites now so we cannot just "force" a default change like that, particularly when it comes to BCU itself. This is why everything we added with NSL went in as an option, which also makes the theme more flexible.

[Info 3bits]

Closed as not applied in this version