Skip to content

Fix for static student gradebook alignment#287

Closed
gjb2048 wants to merge 1 commit intobmbrands:masterfrom
gjb2048:master_static_student_gradebook
Closed

Fix for static student gradebook alignment#287
gjb2048 wants to merge 1 commit intobmbrands:masterfrom
gjb2048:master_static_student_gradebook

Conversation

@gjb2048
Copy link
Copy Markdown
Collaborator

@gjb2048 gjb2048 commented Jun 6, 2014

Fix for https://moodle.org/mod/forum/discuss.php?d=261626 - instead of reducing #user-grades down to 2px which affects table regardless of the core setting 'grade_report_fixedstudents' to bring into alignment, this is now set at 0px such that the scroll bar at the top is completely adjacent as the bottom one is. Then reducing the specific '.left-scroller' (the padding on the contained table has no effect) down by 2px brings everything into alignment when 'grade_report_fixedstudents' is set.

@bmbrands
Copy link
Copy Markdown
Owner

Is this required for BS3? from the forum post it looks like a theme Elegance issue.

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Jun 24, 2014

It is as Elegance is Bs3 based and code is identical.

Sent from my iPod in deep space...

On 24 Jun 2014, at 09:56, Bas Brands notifications@github.com wrote:

Is this required for BS3? from the forum post it looks like a theme Elegance issue.


Reply to this email directly or view it on GitHub.

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Jul 8, 2014

Tis a Bootstrap issue too :)

@bmbrands
Copy link
Copy Markdown
Owner

Hmm I can't see any misalignment on the grade report. Perhaps browser specific?

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 17, 2014

Ah ha! Have you turned this setting on: https://moodle.org/mod/forum/discuss.php?d=261626#p1133556 Bas?

@bmbrands
Copy link
Copy Markdown
Owner

Oh missed this reply. Yes I did have the fixed student column and could not notice any issues. Could you check again too?

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 21, 2014

Hi Bas,

Just tested on current master and seems fine:

2014-08-21 14_01_01-moo27_ administration_ grades_ report settings_ grader report
2014-08-21 14_01_33-grades_ view

G

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 21, 2014

P.S. On Chrome. Not tested elsewhere!

Lunch, so will test on IE after.

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 21, 2014

Long lunch!....

2014-08-21 17_37_03-grades_ view - internet explorer

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 21, 2014

2014-08-21 17_39_41-grades_ view

@bmbrands
Copy link
Copy Markdown
Owner

So we are all good? Thanks a lot for testing!!

@gjb2048
Copy link
Copy Markdown
Collaborator Author

gjb2048 commented Aug 21, 2014

Indeed! Until its broken again!...

gjb2048/moodle-theme_essential#175
gjb2048/moodle-theme_essential#189

In the gradebook, nobody can hear you scream! ;)

@bmbrands bmbrands closed this Oct 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants