Thanks for reporting that and providing a patch, Nadav.

Marina: Would you be able to peer review this?

looks fine for me. Thanks

@Marina, please see if you can pass it forward

Rebased over latest 1-11-2012 master branch
And ready for integration ( I think )

Hi Nadav,
sorry it took long, I was on holidays. Code looks fine to me. Passing.
Thank you
Marina

Rebased over latest 12-11-2012 master and Moodle 23 stable branches

Marina, thanks! Please see if you can change the status and push it forward.

should cherry-pick without problems to all versions (2.4 - master). Thanks

Marina Glancy, I have rebased this fix onto latest (16/8/2013) master.
Moodle 2.3 version is not relevant anymore.
It's a minor change. I think it can be cherry picked into Moodle 2.4/5 easily.
Please see if you can move this forward.

It's already waiting for integration Nadav. Thanks

Integrated to master, 25 and 24 - thanks Nadav.

Ok, so I didn't really see any difference between the two. The 'X' rendered on the next line below the number of points in both cases. I have attached two screenshots.

The first two screenshots are taken in <= Moodle 2.3 where this was an issue but it no longer seems to be. However, in stable master (note: not integration) the 'x' appears outside of the box (screenshot attached). I am not sure if this issue resolves that or if it's another.

Mark Nelson, It seems you are testing it on theme/clean. Were this issue was actually relates to theme/standard.
I should have been more clear in the test instruction. (updated)

Last developer meeting I asked Martin if theme/clean is the new standard theme in Moodle 2.5/6 and the answer was: No.

Obviously, It should also look good on theme/clean. but it was originally meant for theme/standard.

Hi Nadav, I attached two more screenshots in master comparing stable to integration using the standard theme and did not see any differences. Am I missing something?

Hi Mark, You are right. I have just tested it and the fix seem irrelevant any more. It might have been useful for long time ago, with Moodle version 2.3 and master branch at that time.)

Thank you for all your time, testing this issue!

I am closing it.

Revverting and reopening this then

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

Closing this as it is not a bug in any of the supported versions (Only security issues are pushed into 2.3). As Nadav said, it was when this issue was created but is not now.