Skip to content
This repository has been archived by the owner on Oct 30, 2021. It is now read-only.

Update error.php #25

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

rgmears
Copy link

@rgmears rgmears commented Oct 26, 2014

Changes start at line 154.
This gets the main menu displaying correctly but does not fix two other things:
sticky footer (I don't know why the CSS is not applied)
loading of a module within another module. (Tried loadmodule and loadposition to no avail; perhaps because I am using a custom html module for the footer ... the module class is not picked up either.)
As well, I have not figured out how to make the button work for displaying the menu in tablet view.

You can see my custom application here: http://silverplains.ca/floa.html

Changes start at line 154.
This gets the main menu displaying correctly but does not fix two other things:
sticky footer (I don't know why the CSS is not applied)
loading of a module within another module. (Tried loadmodule and loadposition to no avail; perhaps because I am using a custom html module for the footer ... the module class is not picked up either.)
As well, I have not figured out how to make the button work for displaying the menu in tablet view.

You can see my custom application here: http://silverplains.ca/floa.html
<div class="navbar navbar-inverse navbar-fixed-top">
<div class="navbar-inner">
<div class="container">
<a class="brand" href="<?php echo $this->baseurl; ?>"><?php echo htmlspecialchars($app->getCfg('sitename')) ?></a>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't we also want the hamburger icon (https://github.com/construct-framework/bootstruct/blob/develop/index.php#L64-68) before here as well?

@rgmears
Copy link
Author

rgmears commented Oct 27, 2014

Definitely! I just couldn't figure out how to make it work. On my site, when I included it, it displayed in the middle of the navigation band, over top of everything else, instead of off to the right (as in the rest of the site). Plus nothing happened when I clicked on it. So, I included a link to the site map in the text, instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants