Page MenuHomePhabricator

Investigate inlining page specific CSS into PCS HTML layer
Open, NormalPublic

Description

Per the discussion at the Services Sync:

Most CSS will be efficiently handled by the clients…

  • Marvin can do this with isomorphic JS so that it will be inlined on first load, but then handled by the client on subsequent loads.
  • Apps can pack this in and/or download it separately

But for CSS specific to a template or page, it may be best to inline this in the content. We should see if this is viable and check out implications for the clients.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 28 2017, 8:41 PM
Jhernandez lowered the priority of this task from High to Normal.Aug 10 2018, 12:21 PM
Jhernandez added a subscriber: Jhernandez.

Lowering to Normal, we can revisit this once we are further along the integration with apps.

We'd have to do some performance testing of the rendering without inlining and with it, and the size considerations for the cached objects. If it seems worth the tradeoff then, we'd proceed to looking into implementation.