If deferred CSS execution is enabled, and the theme does not include any inline critical CSS, there's a significant reflow which can be disconcerting to content editors.

Given that most admin themes don't include inline CSS, I'd suggest defaulting to not defer CSS in the admin theme.

CommentFileSizeAuthor
#7 make_css_defer-2623224-7.patch5.59 KBNickDickinsonWilde
#2 make_css_defer-2623224-2.patch1.97 KBmalcomio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

malcomio created an issue. See original summary.

malcomio’s picture

malcomio CreditAttribution: malcomio commented
Status: Active » Needs review
FileSize
make_css_defer-2623224-2.patch1.97 KB
mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Needs review » Reviewed & tested by the community

Looks like the patch works; will commit soon.

mikeytown2’s picture

mikeytown2 CreditAttribution: mikeytown2 commented
Status: Reviewed & tested by the community » Fixed

Patch committed!

  • mikeytown2 committed 0b48f28 on 7.x-2.x authored by malcomio 
    Issue #2623224 by malcomio: Make CSS defer configurable for admin theme...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

NickDickinsonWilde’s picture

NickDickinsonWilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
CreditAttribution: NickDickinsonWilde commented
Version: 7.x-2.x-dev » 8.x-2.x-dev
Assigned: Unassigned » NickDickinsonWilde
Status: Closed (fixed) » Patch (to be ported)
FileSize
make_css_defer-2623224-7.patch5.59 KB

porting to 8.x-2.x

NickDickinsonWilde’s picture

NickDickinsonWilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
CreditAttribution: NickDickinsonWilde commented
Assigned: NickDickinsonWilde » Unassigned
Status: Patch (to be ported) » Closed (fixed)