@Dietmar

I've just set this up and added you as a watcher.
Please let me have some feedback, from the files I sent you for this fix, when you have time. There is no big rush.

Cheers
Mary

Hi, Mary.

I've just triaged this, but please keep working on it.

Hi Mary,

I tested your files and performance info is now shown in the footer. Great!

There's one minor issue left as I have different layouts on different pages.
Please have look at the attached images.

Perhaps it would make sense to add something like

#page-footer .performanceinfo span

or

#page-footer .performanceinfo span

to core.css for a consistent layout!

Cheers
Dietmar

Hi Dietmar,

Thanks for this suggestion, I'll test it later when I am fixing the theme, which I plan to do after dinner this evening! Perhaps with a bottle of vino too!

Cheers
Mary

@Dietmar,

This took me longer than I thought it would...because halfway through doing the changes, I realised that the css I was adding for the performance info et al, should actually be styled by Base theme. As a result ended up restyling the footer again with a better results this time and less css.

Cheers
Mary

Hi Mary,

I've just been looking at this now.
This one certainly looks like it should be backported to 22 and 21.
I've attempted to cherry-pick however it does not apply cleanly.
Could you please confirm this needs to be backported and produce branches for me.

Cheers
Sam

@ Sam,
Yes please, this needs back porting to 21 & 22.
Now M21 & M22 branches are in place are good to go.

Cheers
Mary

Thanks for cleaning things up Mary, this has been integrated now

Note I've performed one bulk whitespace fix to the theme on 21_STABLE, 22_STABLE and master, as far as this was adding new incorrect whitespace.

It's all there now, looks fine

Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many.

Nah, joking, many thanks! Closing this a fixed, ciao