Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.2.3
    • Component/s: Themes
    • Labels:

      Description

      I was surprised to find in FW/styles/core.css about line 113

      .loginbox .loginform .form-label {
          float: none;
          width: 100%;
          margin: 0 auto;
          float: left;
          text-align: right;
          width: 40%;
      }

      so I deleted useless roles

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            daniss Daniele Cordella added a comment -

            I apologise for not have seeing this silly issue during last patch

            Show
            daniss Daniele Cordella added a comment - I apologise for not have seeing this silly issue during last patch
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Some hours ago...

            the main moodle.git repository has 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 - Some hours ago... the main moodle.git repository has 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
            daniss Daniele Cordella added a comment -

            Eloy, it so risk free this patch that I do not feel I am bad if I ask you to include it without integration review process. Ok, integration review is a crucial step but this patch is really "plain water".

            – Posted from Bugbox for iPhone

            Show
            daniss Daniele Cordella added a comment - Eloy, it so risk free this patch that I do not feel I am bad if I ask you to include it without integration review process. Ok, integration review is a crucial step but this patch is really "plain water". – Posted from Bugbox for iPhone
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Daniele, changes were 100% straight forward and have been integrated now.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Daniele, changes were 100% straight forward and have been integrated now.
            Hide
            abgreeve Adrian Greeve added a comment -

            I know that there are no test instructions, but I loaded up the login in form and inspected the page with firebug. Everything looks good.
            Thanks.

            Show
            abgreeve Adrian Greeve added a comment - I know that there are no test instructions, but I loaded up the login in form and inspected the page with firebug. Everything looks good. Thanks.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            FCT (fixed, closing, thanks). Ciao

            "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!"
            ~ Benjamin Disraeli

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - FCT (fixed, closing, thanks). Ciao "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!" ~ Benjamin Disraeli

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12