moodle-theme_essential icon indicating copy to clipboard operation
moodle-theme_essential copied to clipboard

Grade report alignment horizontal scroll on small screen size

Open mnods opened this issue 6 years ago • 8 comments

using 3.5.1.0 and 3.4.1.2

i recall this being an issue in an earlier version.

on smaller screen size in grader report, (grade/report/grader/index.php) using horiz scroll with a fair number of students, the column that sticks (div with classes "floater sideonly floating") is vertically misaligned, making it very hard to grade properly, especially with a large class.

altering the inline css to top: 88px;

seems to fix it.

seems ok in clean.

mnods avatar Nov 06 '18 05:11 mnods

i've found with this issue that a page refresh realigns the floater column. haven't tracked down the cause though.

mnods avatar Nov 07 '18 02:11 mnods

note- misalignment happens at less than 979px

mnods avatar Nov 07 '18 02:11 mnods

Ok 'altering the inline css to top: 88px;' won't work because that value is generated (and changed) by the underlying JavaScript. The issue is caused by the core JavaScript being written for the specifics of the themes in core Moodle and thus is not adaptable / flexible enough to cope with changes in contributed themes - its written in a constraining way. The solution is to disable that JS (somehow) and use theme specific JS to implement the same functionality or find another solution to the issue of not seeing the table headers when there are a large number of columns. This is not 'trivial' as I think the markup output is difficult to override.

gjb2048 avatar Nov 07 '18 10:11 gjb2048

yes gareth, i get that i can't change the inline generated css, just wondered if there's a reasonably simple adjustment that can be made to the css for below 979px that is causing the extra height for that floating column calculation. the height calc looks fixed to me. not easy i can see. weird that a refresh resets it.

mnods avatar Nov 07 '18 10:11 mnods

I've previously spent 'days' on this! I'll look again when I get a moment. This is not easy!!

gjb2048 avatar Nov 07 '18 10:11 gjb2048

given the way it's set up and the difficulty involved, i wonder whether the grade report should really be stripped of theme styles in the same way as the assignment grading page?

mnods avatar Nov 12 '18 03:11 mnods

Possibly.

gjb2048 avatar Nov 12 '18 12:11 gjb2048

This one is difficult, so going to leave for now.

gjb2048 avatar Nov 25 '18 15:11 gjb2048