Basis: Design and theme the Maintenance page

[klonos]

Current Basis

Bartik:

Seven:

Previous Basis:

---

PR: backdrop/backdrop#2511
Milestone candidate tag for 1.12.3 meanwhile 1.12.7.

[Graham-72]

If we are in the process of loading a full update of Backdrop, is it reasonable to expect a fully-themed page here?

[klonos]

Nope, not a fully themed one - I was thinking at least just the basics like centering the text vertically and horizontally and keeping the fonts and font sizes/formatting. Perhaps consider also rendering the header ribbon with the logo, site name and slogan. Basically, the minimum required for people to recognize the brand of the site.

Bartik does minimal things I believe:

https://github.com/backdrop/backdrop/blob/1.x/core/themes/bartik/template.php#L13
https://github.com/backdrop/backdrop/blob/1.x/core/themes/bartik/templates/maintenance-page.tpl.php
https://github.com/backdrop/backdrop/blob/1.x/core/themes/bartik/css/maintenance-page.css

Keep in mind that the intention here is to provide themers with something that they can easily override.

[klonos]

...the point is to be retaining a "professional" look when the site is in maintenance mode. As it is now, site visitors will most likely have the impression that the site is "broken" (instead of being in scheduled/intentional maintenance mode that is).

[klonos]

https://github.com/backdrop/backdrop/blob/1.x/core/themes/basis/template.php#L10

/**
 * Implements hook_preprocess_page().
 *
 * @see maintenance_page.tpl.php
 */
function basis_preprocess_page(&$variables) {

The @see in the docblock in the main template file points to a maintenance page template that Basis does not include. Perhaps a copy-paste mistake or omission while @wesruv was coding Basis?

[klonos]

...even in maintenance mode, the login and password reset pages are themed, which is a good thing IMO. We just need to address a couple of issues: #2529

[klonos]

...updated issue summary to include a screenshot of the maintenance page for Seven (which is doing a great job BTW).

[klonos]

...I think I'm gonna take a shot at this a bit later, after I get some 💤 😄

[jenlampton]

@klonos can you please create a separate issue for The @see in the docblock in the main template file?

From the screenshots it does look like the page is using the correct theme. The correct fonts from Basis are appearing, for example. I think the problem is this...

As it is now, site visitors will most likely have the impression that the site is "broken" (instead of being in scheduled/intentional maintenance mode that is).

That sounds like a design issue (and an important one!) but not really a bug.

Also... Pushing this to the 1.6.2 milestone since there's no PR and we are releasing 1.6.1 tonight.