Moodle
  1. Moodle
  2. MDL-26228

Grade full view cuts off viewable area

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.0.3
    • Component/s: Gradebook
    • Labels:
    • Environment:
      php 5.2.10 . RHEL 5.4
    • Database:
      MySQL
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      15946

      Description

      When looking at the grade book in Full view the columns are cut when viewing in smaller browser windows.

      Solution add overflow:auto to Line 18 of theme/base/style/grade.css
      .gradetreebox

      {margin-top:10px; overflow:auto; }

        Activity

        Hide
        Andrew Davis added a comment -

        There was similar css already there but it was in the standard theme rather than the base theme. Ive moved it into base.

        repo: git://github.com/andyjdavis/moodle.git
        branch: MDL-26228_grade_full_view
        diff: https://github.com/andyjdavis/moodle/compare/master...MDL-26228_grade_full_view

        Show
        Andrew Davis added a comment - There was similar css already there but it was in the standard theme rather than the base theme. Ive moved it into base. repo: git://github.com/andyjdavis/moodle.git branch: MDL-26228 _grade_full_view diff: https://github.com/andyjdavis/moodle/compare/master...MDL-26228_grade_full_view
        Hide
        Andrew Davis added a comment -

        PULL-364

        Show
        Andrew Davis added a comment - PULL-364
        Hide
        Petr Škoda added a comment -

        Reopening because there was no feedback for the failed test in PULL request.

        Show
        Petr Škoda added a comment - Reopening because there was no feedback for the failed test in PULL request.
        Hide
        Andrew Davis added a comment -

        Responded in PULL-364. Ive now raised PULL-372.

        Show
        Andrew Davis added a comment - Responded in PULL-364. Ive now raised PULL-372.
        Hide
        Petr Škoda added a comment -

        Reopening due to bogus author in git commit, sorry.

        Show
        Petr Škoda added a comment - Reopening due to bogus author in git commit, sorry.
        Hide
        Andrew Davis added a comment -

        PULL-399

        Show
        Andrew Davis added a comment - PULL-399
        Hide
        Eloy Lafuente (stronk7) added a comment - - edited

        Related to this, always I see those reports under standard... I think current 70% is waste of space. So I would do something like:

        diff --git a/theme/standard/style/grade.css b/theme/standard/style/grade.css
        index 1f9c2df..81579f4 100644
        --- a/theme/standard/style/grade.css
        +++ b/theme/standard/style/grade.css
        @@ -18,12 +18,11 @@ td.grade div.overridden {background-color: #DDDDDD;}
         .path-grade-report-grader .graderreportoverlay {background-color:#EEEEEE;border:1px solid black;padding:10px;}
         
         .path-grade-edit-tree .moving {background-color: #E8EEF7;}
        -.path-grade-edit-tree .gradetreebox {width:70%;padding-bottom:15px;overflow-x:auto;overflow-y:hidden;}
        +.path-grade-edit-tree .gradetreebox {padding-bottom:15px;overflow-x:auto;overflow-y:hidden;}
         
         .path-grade-edit-scale .scale_options,
         #page-admin-grade-edit-scale-index .scale_options {font-size: 0.8em;}
         
        -.gradetreebox {margin:10px auto;width:90%;}
        

        Perhaps taking out the whole box could have sense too. Other reports (grader) don't use it at all.

        Just one comment, apart from current integration of PULL-399.

        Show
        Eloy Lafuente (stronk7) added a comment - - edited Related to this, always I see those reports under standard... I think current 70% is waste of space. So I would do something like: diff --git a/theme/standard/style/grade.css b/theme/standard/style/grade.css index 1f9c2df..81579f4 100644 --- a/theme/standard/style/grade.css +++ b/theme/standard/style/grade.css @@ -18,12 +18,11 @@ td.grade div.overridden {background-color: #DDDDDD;} .path-grade-report-grader .graderreportoverlay {background-color:#EEEEEE;border:1px solid black;padding:10px;} .path-grade-edit-tree .moving {background-color: #E8EEF7;} -.path-grade-edit-tree .gradetreebox {width:70%;padding-bottom:15px;overflow-x:auto;overflow-y:hidden;} +.path-grade-edit-tree .gradetreebox {padding-bottom:15px;overflow-x:auto;overflow-y:hidden;} .path-grade-edit-scale .scale_options, #page-admin-grade-edit-scale-index .scale_options {font-size: 0.8em;} -.gradetreebox {margin:10px auto;width:90%;} Perhaps taking out the whole box could have sense too. Other reports (grader) don't use it at all. Just one comment, apart from current integration of PULL-399.
        Hide
        Helen Foster added a comment -

        Scott, thanks for your report and Andrew, thanks for fixing.

        The issue is fixed in this week's 2.0.2+.

        Show
        Helen Foster added a comment - Scott, thanks for your report and Andrew, thanks for fixing. The issue is fixed in this week's 2.0.2+.

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: