Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-42853

Wrong alignment for modules on frontpage in RTL in Clean

    XMLWordPrintable

Details

    • MOODLE_25_STABLE, MOODLE_26_STABLE
    • MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
    • MDL-42853-master
    • Hide
      1. Test in both Standard and Clean:
      2. Add some modules to the frontpage
      3. On the front page, make sure there is no unexpected margin/padding to the section in LTR or RTL.
      4. For master, make sure that the padding/margin between the "Edit title" icon and the activity name is consistent between RTL and LTR (and looks good) - front page + course page
      Show
      Test in both Standard and Clean: Add some modules to the frontpage On the front page, make sure there is no unexpected margin/padding to the section in LTR or RTL. For master, make sure that the padding/margin between the "Edit title" icon and the activity name is consistent between RTL and LTR (and looks good) - front page + course page

    Description

      ul, ol {    margin: 0 0 10px 25px;    padding: 0;}
      		 
      .site-topic ul.section, .course-content ul.section {    margin: 1em;}

      The second statement balances the margins and no RTL rule is needed.
      But it does not apply to frontpage because it should be 'sitetopic' and not 'site-topic'

      which makes modules lists on frontpage be padded on the wrong side in RTL.

      Also the padding/margin of the "Edit title" icon is on the wrong size.

      Attachments

        1. ss3.png
          ss3.png
          6 kB
        2. ss4.png
          ss4.png
          6 kB

        Issue Links

          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-42851 Course listings in Clean RTL

          • Major - Major loss of function, incorrect output
          • Closed

          Activity

            People

              fred Frédéric Massart
              marina Marina Glancy
              Marina Glancy Marina Glancy
              Sam Hemelryk Sam Hemelryk
              Michael de Raadt Michael de Raadt
              Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona), Huong Nguyen, Bas Brands, Mathew May
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                13/Jan/14