INN / umbrella-ghn

Umbrella repository for Georgia Health News
GNU General Public License v2.0
0 stars 1 forks source link

Move sidebar content below everything else on mobile homepage #8

Open MirandaEcho opened 4 years ago

benlk commented 4 years ago

The homepage is built with WPBakery Page Builder. The sidebar is part of a row that comes before the sliders below the full-width element on the homepage bottom. Here's a very zoomed-out screenshot of what that looks like in practice:

Screen Shot 2020-04-03 at 16 33 43

benlk commented 4 years ago

The page builder does not appear to have a Grid block.

Options:

  1. Duplicate the existing sidebar; place the clone below the sliders. Set the existing sidebar to show not on mobile; set the clone to show only on mobile.
  2. Move the sliders at the bottom of the homepage into the main column, so that the existing sidebar pulls alongside them. No duplication within the editor, but now the sliders aren't full width.
  3. Adding styles or JS that don't respect how the page-builder works, which would cut the sidebar from its present position and paste it elsewhere, when on mobile. This would not be configurable by them.

Which would GHN prefer?

benlk commented 4 years ago

1 or 2 are easier for us and them, and require the least time. They also keep the entire context of the displayed page in the pagebuilder, so that if they edit their homepage themselves that'll be all they need to know.

Let's rule out 3.

The difference between 1 and 2 is whether they want to keep their current homepage layout on desktop. They probably have opinions on that.