Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Fixed table

...

<yhahn>

EvanDonovan: it is checked server side for rather trivial stuff

 

<EvanDonovan>

yhahn: if it were not present would that be bad.  

<yhahn>

well not trivial but I believe you could exclude it if it is an issue

 

<yhahn>

it basically sets the initial css classes for the toolbar/body

 

<EvanDonovan>

b/c if i strip it in Varnish VCL then it won't be there  

<EvanDonovan>

ah ok  

<EvanDonovan>

i was hoping that would only happen client-side  

<yhahn>

it's there to prevent slow-load js flicker  

<EvanDonovan>

i will not strip it if the server-side checks are done in the theming layer

 

<EvanDonovan>

yeah, that makes sense

 

<EvanDonovan>

i will keep it then

 

<EvanDonovan>

most of the time it will be moot point  

<yhahn>

cools  

<EvanDonovan>

b/c most users with access to admin toolbar will not be anonymous

...