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

Fix CSS errors in Magazine theme

    Details

    • Testing Instructions:
      Hide
      1. Select Magazine theme.
      2. Select Hebrew or other RTL Language in Lang menu.
      3. Set Frontpage to show Combo list (Course/category tree).
      4. TEST that when a category list containing a number of courses is collapsed that the arrow is visible and centre right of the list: see attached image (MDL-38380.jpg).
      Show
      Select Magazine theme. Select Hebrew or other RTL Language in Lang menu. Set Frontpage to show Combo list (Course/category tree). TEST that when a category list containing a number of courses is collapsed that the arrow is visible and centre right of the list: see attached image ( MDL-38380 .jpg).
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull Master Branch:

      Description

      .dir-rtl.jsenabled .course_category_tree .category.with_children.collapsed .category_label

      { backkground-position: right center !important; }

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            lazydaisy Mary Evans added a comment -

            @Nadav: Just added you here as watcher as this may be affecting the view of this from the RTL commuinity. I'll be fixing this this evening.

            Show
            lazydaisy Mary Evans added a comment - @Nadav: Just added you here as watcher as this may be affecting the view of this from the RTL commuinity. I'll be fixing this this evening.
            Hide
            nadavkav Nadav Kavalerchik added a comment -

            Thanks

            Show
            nadavkav Nadav Kavalerchik added a comment - Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            lazydaisy Mary Evans added a comment -

            REBASED ALL BRANCHES:

            Show
            lazydaisy Mary Evans added a comment - REBASED ALL BRANCHES:
            Hide
            lazydaisy Mary Evans added a comment -

            Updated and tried to rebase and ended up with conflicts. So rebuilt merged/squashed and hopefully got back on track.

            Show
            lazydaisy Mary Evans added a comment - Updated and tried to rebase and ended up with conflicts. So rebuilt merged/squashed and hopefully got back on track.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            I'm sorry but this needs some testing instructions, at least pointing to places where changes have been done.

            For all branches, it's the "kk" fix, but in master there are a bunch of changes involving !important. It should be verified (tested) that they are really not needed.

            Finally, please, don't mix fixes with code cleanup. It does things really hard to review. Create one "cleanup" issue and perform ONLY cleanup changes on it.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - I'm sorry but this needs some testing instructions, at least pointing to places where changes have been done. For all branches, it's the "kk" fix, but in master there are a bunch of changes involving !important. It should be verified (tested) that they are really not needed. Finally, please, don't mix fixes with code cleanup. It does things really hard to review. Create one "cleanup" issue and perform ONLY cleanup changes on it. Ciao
            Hide
            cibot CiBoT added a comment -

            Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

            Show
            cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
            Hide
            damyon Damyon Wiese added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            Thanks!

            Show
            damyon Damyon Wiese added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. Thanks!
            Hide
            lazydaisy Mary Evans added a comment -

            Oh...forgot to say this has been rebased.

            Show
            lazydaisy Mary Evans added a comment - Oh...forgot to say this has been rebased.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks Mary,

            This has been integrated to 23, 24 and master.

            Show
            damyon Damyon Wiese added a comment - Thanks Mary, This has been integrated to 23, 24 and master.
            Hide
            dmonllao David Monllaó added a comment -

            It passes, all according to the screenshot.

            I've only noticed the RTL expand icon style is different from collapse icon style and LTR expand icon style.

            Show
            dmonllao David Monllaó added a comment - It passes, all according to the screenshot. I've only noticed the RTL expand icon style is different from collapse icon style and LTR expand icon style.
            Hide
            lazydaisy Mary Evans added a comment -

            Oh it is isn't it! In that case I better fix that too. Thanks David.

            Show
            lazydaisy Mary Evans added a comment - Oh it is isn't it! In that case I better fix that too. Thanks David.
            Hide
            damyon Damyon Wiese added a comment -

            Thanks for your hard work. This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).

            Show
            damyon Damyon Wiese added a comment - Thanks for your hard work. This issue has been integrated upstream and is now available via git (and in some hours, via mirrors and downloads).

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  13/May/13