Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6.1
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide

      This needs 'clean' as theme and RTL language.

      1. Set front-page to display: list of courses, combo list, and list of categories.
      2. Make sure the layout and icons are position correctly as in standard theme (or 2.5 rtl in clean theme).
      Show
      This needs 'clean' as theme and RTL language. Set front-page to display: list of courses, combo list, and list of categories. Make sure the layout and icons are position correctly as in standard theme (or 2.5 rtl in clean theme).
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      Course listings don't look good in Clean theme in RTL, see screenshots

        Gliffy Diagrams

          Attachments

          1. clean_ltr2.png
            clean_ltr2.png
            14 kB
          2. clean_rtl1.png
            clean_rtl1.png
            33 kB
          3. clean_rtl2.png
            clean_rtl2.png
            14 kB
          4. standard_ltr2.png
            standard_ltr2.png
            12 kB
          5. standard_rtl1.png
            standard_rtl1.png
            19 kB
          6. standard_rtl2.png
            standard_rtl2.png
            12 kB

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              I can confirm that this doesn't appear to be a problem in 2.5.

              Show
              salvetore Michael de Raadt added a comment - I can confirm that this doesn't appear to be a problem in 2.5.
              Hide
              marina Marina Glancy added a comment -

              I'm afraid it may be related to headings issues, Rossiani Wijaya can you please look at it?

              Show
              marina Marina Glancy added a comment - I'm afraid it may be related to headings issues, Rossiani Wijaya can you please look at it?
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Sure.

              I'm assigning this issue to myself.

              Show
              rwijaya Rossiani Wijaya added a comment - Sure. I'm assigning this issue to myself.
              Hide
              jerome Jérôme Mouneyrac added a comment -

              I tested, it does fix all issues I could see in the screenshots:

              • the icons been displayed next to the text are now displayed on the end of the row
              • the teacher info and summary are inversed
              Show
              jerome Jérôme Mouneyrac added a comment - I tested, it does fix all issues I could see in the screenshots: the icons been displayed next to the text are now displayed on the end of the row the teacher info and summary are inversed
              Hide
              poltawski Dan Poltawski added a comment -

              Integrated to master and 26 - thanks Rosie

              Show
              poltawski Dan Poltawski added a comment - Integrated to master and 26 - thanks Rosie
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Tested and passed thanks Rosie

              Show
              samhemelryk Sam Hemelryk added a comment - Tested and passed thanks Rosie
              Hide
              poltawski Dan Poltawski added a comment -

              Congratulations, this change has now made its way upstream. Thanks for your contribution!

              “ Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. ” - Rick Osborne

              Show
              poltawski Dan Poltawski added a comment - Congratulations, this change has now made its way upstream. Thanks for your contribution! “ Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. ” - Rick Osborne

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    13/Jan/14