Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Bootstrap Site maintenance page looks as you can see in the following screenshot:
Proposed resolution
Have basic themed Site maintenance page.
As you can see in the following screenshot:
Remaining tasks
- No tasks
User interface changes
- Site Logo at the center.
- Maintenance message.
- Responsive view of the Site maintenance page.
Comment | File | Size | Author |
---|---|---|---|
#33 | 2865975-33.patch | 4.25 KB | markhalliwell |
Comments
Comment #2
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #3
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #4
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #5
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThis is the patch to have the basic themed Site maintenance page.
Comment #6
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #7
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #8
markhalliwellI'd rather this not be a panel. A truly "basic" implementation would be just like the normal page: navbar with logo + site name/slogan, a page header and content.
This makes the assumption that the site/content is copyrighted. I'd rather this project not make these assumptions. This can easily be added in a sub-theme as needed.
Comment #9
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #10
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThank you Mark,
Noted; I thought that would be your points in the feedback :)
Following with your directions.
A new patch file
Comment #11
markhalliwellThe database may not be reachable during maintenance. Theme settings will need defaults in case these aren't provided.
Replace the navigation region with the branding. This is normally a block, but this isn't available in maintenance mode.
It should be in the navbar header to mimc the normal site.
There's no need for this since there are no menu items.
These are unnecessary. Just set to
col-sm-12
There's no need for superfluous regions.
Comment #12
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedDo you think having an extend will help?
{% extends "page.html.twig" %}
It's your call Mark! I'm happy to have it with the basic style. Not sure if extend will do that job.
I have a new patch file
If not connected to the database. In Drupal 7 I used to use something as the following:
Trying to have that in Drupal 8 or something around.
Comment #13
markhalliwellDon't center the text please.
Comment #14
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThanks a lot for following on this issue.
I made the change, and it will look as you can see in the attached image
Attached the patch file.
Comment #15
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #16
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #17
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #18
markhalliwellPlease remove the
primary-column
class as it is not needed.The navbar classes should be added to the header/branding containers to make it look decent.
Comment #19
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedNew patch for 8.x-3.x
And patch for 7.x-3.x
And only for D7.x-3.x: if the database was not connected to the database. db_is_active == false
Comment #20
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #21
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #22
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #23
markhalliwellNavbar and branding both need to mimic what already exist:
https://drupal-bootstrap.org/api/bootstrap/templates%21system%21page.htm...
https://drupal-bootstrap.org/api/bootstrap/templates%21block%21block--sy...
It should look like what it does out of the box:
Comment #24
xeniksp CreditAttribution: xeniksp commentedi copied the values from the https://drupal-bootstrap.org/api/bootstrap/templates%21block%21block--system-branding-block.html.twig/8#source.20,25 and i edit the patch .
Comment #25
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedHappy with that.
Comment #26
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #27
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #29
markhalliwell7.x-3.x still needs an updated backported patch like #24.
Comment #31
xeniksp CreditAttribution: xeniksp commentedHere is an updated patch for 7.x-3.x
Comment #32
xeniksp CreditAttribution: xeniksp commentedI removed the global $theme because i think it is not needed and i replaced the $content_column_classes with .col-sm-12
Comment #33
markhalliwell