Sal Ferrarello
Sal Ferrarello
Yeah, I feel like we've been supporting XHTML when we think of it but not making it a priority. I'm in favor of stripping XHTML from the theme and making...
I've turned this into an actionable item in #155
@bryanwillis As I look at the changes involved, I'm revisiting this question. I agree that we could change ``` $nav = genesis_markup( array( 'html5' => '', 'xhtml' => '', 'context'...
I'm still wrapping my head around this code but a couple of quick notes so I don't need to remember them for later. ## PHP Warning I'm getting the warning...
Looking at the Genesis code in `lib/functions/menu.php`, it looks like the filters `genesis_do_nav` and `genesis_do_subnav` are still in place for backwards compatibility. So instead it looks like we should be...
I believe the Bootstrap nav only supports a depth of 2. So, I think we should change this value from 3 to 2. ``` $args['depth'] = 3; ```
@bryanwillis Cool, thanks for sharing this.
With this code change I am seeing some differences in the output nav code. ## For Example ### Old Output ``` About ``` ### New Output ``` About ``` ##...
See branch `change-col-other-layouts-119` for proposed changes.
Ok, I've been reviewing my previous preposed changes, which looked like this for **sidebar-content-sidebar**  I think this is simply too narrow for the content column...