INN / inn

WordPress child theme for INN.org. Requires Largo parent theme (https://github.com/INN/largo)
https://inn.org
GNU General Public License v2.0
2 stars 2 forks source link

Copy Header Widget area to mobile menu; remove "Hire Us" section from homepage #132

Closed benlk closed 4 years ago

benlk commented 4 years ago

Changes

This pull request makes the following changes:

Here's the current Header Widget setup in the menu:

Screen Shot 2020-07-29 at 21 57 20

Left: the homepage without the "Hire Us" block

Screen Shot 2020-07-29 at 22 52 32

Why

Resolves https://github.com/INN/umbrella-inndev/issues/169 and resolves https://github.com/INN/umbrella-inndev/issues/168

Testing/Questions

Features that this PR affects:

Questions that need to be answered before merging:

Steps to test this PR:

  1. View the homepage to make sure that the "Hire us" section is gone.
  2. On a narrow viewport where the sticky menu appears with the mobile hamburger menu, open it and scroll to the bottom to see the appearance of widgets in the "Header Widget" area. It's presently styled with the assumption that whatever shows up there is a text widget with buttons, or similar.
benlk commented 4 years ago

:deployed: to staging for review: https://inndev.staging.wpengine.com/

benlk commented 4 years ago

Approved at https://innorg.slack.com/archives/G0DSPDQ00/p1596132405000600