Questions that need to be answered before merging:
[x] Is this PR targeting the correct branch in this repository?
[ ] Do we want to try to merge header.php and header-rezone.php somehow, like for example having header-rezone.php load header.php? They're not that different, with the exception of:
the addition of a body class for rezone
the .rezone-header section containing the menu
js for the rezone menu
removal of the global nav, which we aren't using anyways
the addition in header.php of improved support for things like WordPress' title tag and wp_body_open
Steps to test this PR:
Visit the localhost equivalents of any of those above-mentioned URLs. Alternately, search your wp_post_meta table for meta_value matching any of the post templates mentioned above, and view those posts.
Changes
This pull request removes the "Mapping the Future" logo from the header that is used on:
page-neighborhoods.php
- https://citylimits.org/zonein/page-communitywire.php
- I'm unsure where this is used; my db copy says post 2988252 which is https://citylimits.org/communitywire-home/ but that's a 404 on prodpage-neighborhood-info.php
- https://citylimits.org/zonein/affordable-housing-101/ as an exampleWhy
For #147 because this logo is old
Testing/Questions
Features that this PR affects:
Questions that need to be answered before merging:
header.php
andheader-rezone.php
somehow, like for example havingheader-rezone.php
loadheader.php
? They're not that different, with the exception of:.rezone-header
section containing the menuheader.php
of improved support for things like WordPress' title tag andwp_body_open
Steps to test this PR:
wp_post_meta
table formeta_value
matching any of the post templates mentioned above, and view those posts.