I've put this on the Stable backlog, but please feel free to work on this as you see fit.

Michael;

Hi John,
The reason the font size was left at 85% is because that was the font-size which was previously set in text.css. It may actually be worth considering taking text.css out all together in Moodle 2.1 and adding it in to the core themes which need it.

Also related to this fix (see MDL-27704) was the problem with Fusion theme, it might be worth taking another look at that theme after you up the font to 108% in Canvas theme, as the fix I added, that of increasing the following values in Fusion, might, in actual fact, not now be needed, so these values will need to be reversed:

#page-site-index #region-header {
/* margin-bottom: 55px; */
margin-bottom: 65px;

#region-header {
/* margin-bottom: 30px; */
margin-bottom: 40px;
}

Mary

I see this as pretty urgent ... Moodle text looks very small now ...

The fix is posted up now.

Mary,

Fusion looks okay now with the font size fix (and with your other changes still in place). So I don't think it will need a 're-fix'. I agree that text.css probably needs to go at some point and the move of font-size (etc.) to #page in canvas was the right call.

John

I've just taken a look at the patch you are adding and it looks OK,
I've set it to 'Waiting for integration review', so hopefully Sam or Eloy will get to see this soon.

Thanks
Mary

Just for confirmation... this also affects 20_STABLE that we have rolled a few hours ago.. is it important enough to re-reroll 20_STABLE again today? Or can wait till next Wednesday?

Also, plz, confirm if it has been already tested, so I can skip that!

I think it is pretty important, as it effects usability across most themes... Whether that is important enough to re-roll 20_STABLE, I can't say...

oki, 2 notes:

1) I've cherry picked your commits because there were a mistake in the issue number.
2) Perhaps it would be interesting, John, to configure your git to have proper fullname and email. I think right now you're the only one not showing that, but "stabijoh". For your consideration.
3) Being important, I'm going to re-roll 20_STABLE and master NOW. Can I rely 100% the fix is ok? I've tried fusion and seems ok (clearly bigger than before the patch).

Integrated, thanks!

Tested with Fusion, looks really better now. That and relying into devs are enough to consider this passed.

Going to re-roll 20_STABLE and master weeklies right now.

Thanks and ciao

Done. Now this is in all git & cvs servers. Next packages will also include the fix. So closing.

Thanks and ciao

Thanks for this Eloy. Sorry I was not about earlier otherwise I would have answered YES it had been tested!

Cheers
Mary

Thanks everyone for fixing this issue. I was thinking I was becoming old and needed glasses, but today found the binarius theme on http://qa.moodle.net/ looking perfect!