May Toolbar resources be moved to plone-logged-in bundle?

Development
1

Currently plone-patterns-toolbar is loaded for anonymous, in https://github.com/plone/Products.CMFPlone/blob/master/Products/CMFPlone/static/plone.less#L8, instead of in https://github.com/plone/Products.CMFPlone/blob/master/Products/CMFPlone/static/plone-logged-in.less

May we move that in order to get a slimmer site for anonymous? I also think it's more intuitive to find it there.

2

+1 - targeting Plone 5.1 only.

How much compressed javascript transmission does this save?

3

Well, toolbar.js is 14.5kb and toolbar.plone.less + variables.less is 19.5kb, making for around 34kb but that's before compiling/minifying/gziping. There's probably also some blocking in browser's time to interact, both for parsing and rendering.

4

Definitely. It should be in the logged in bundle.

5

Great! I created a PR to adddress it https://github.com/plone/Products.CMFPlone/pull/1703

1 Like