@Nadav: Just added you here as watcher as this may be affecting the view of this from the RTL commuinity. I'll be fixing this this evening.

Thanks

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

REBASED ALL BRANCHES:

Updated and tried to rebase and ended up with conflicts. So rebuilt merged/squashed and hopefully got back on track.

I'm sorry but this needs some testing instructions, at least pointing to places where changes have been done.

For all branches, it's the "kk" fix, but in master there are a bunch of changes involving !important. It should be verified (tested) that they are really not needed.

Finally, please, don't mix fixes with code cleanup. It does things really hard to review. Create one "cleanup" issue and perform ONLY cleanup changes on it.

Ciao

Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

Thanks!

Oh...forgot to say this has been rebased.

Thanks Mary,

This has been integrated to 23, 24 and master.

It passes, all according to the screenshot.

I've only noticed the RTL expand icon style is different from collapse icon style and LTR expand icon style.

Oh it is isn't it! In that case I better fix that too. Thanks David.

Thanks for your hard work. This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).