The maintenance page does hide the skip to main content link. This is because system.module.css is no longer added in as I think I was in Drupal 7 afaik the equivalent system css files were added during system_init() when the db was available. They are now added in system_page_build() which is not fired for maintenance pages.

Current maintenance page

Maintenance_page-2.png

CommentFileSizeAuthor
#4 interdiff.txt1.33 KBalansaviolobo
#4 maintenance_page-2022765-4.patch701 bytesalansaviolobo
#1 Maintenance_page_after.png25.24 KBalexpott
#1 2022765.maintanance-visually-hidden.1.patch1.27 KBalexpott
Maintenance_page-2.png39.57 KBalexpott

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2022765.maintanance-visually-hidden.1.patch1.27 KB
new Maintenance_page_after.png25.24 KB

With the patch attached the page looks like this...
Maintenance_page_after.png

amateescu’s picture

amateescu commented

Is there any reason not to move the other two calls to drupal_add_css() with the rest of the bunch?

amateescu’s picture

amateescu commented
Priority: Critical » Normal

Also, a site is not in an unusable state (wsod, fatal error, etc) if that link is not hidden, so I don't think this is critical.. not even major :)

alansaviolobo’s picture

alansaviolobo commented
Issue summary: View changes
StatusFileSize
new maintenance_page-2022765-4.patch701 bytes
new interdiff.txt1.33 KB

the patch didn't seem to make any difference. i guess this issue is already fixed ?

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs review » Closed (duplicate)

This appears to have been fixed along the way. I've confirmed that these CSS files are present on the maintenance page, and hide the "Skip to main content" as expected.