Moodle
  1. Moodle
  2. MDL-32321

Autolink class is missing from "Formal White" theme in Moodle 2.2

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.3
    • Component/s: Themes
    • Labels:
      None
    • Testing Instructions:
      Hide

      Enable filter "Activity names auto-linking".
      Add an activity.
      Write somewhere in moodle pages (let's say in the course description) the name of the just added activity.
      Moodle should automatically create a link to that activity with the autolink class.
      That link is supposed to be underlined to let user understand it is a link and not a simple plain text.

      Test also the last minute change: go to the notification page. The box with admin warning about maturity info should now have a green background color.

      Show
      Enable filter "Activity names auto-linking". Add an activity. Write somewhere in moodle pages (let's say in the course description) the name of the just added activity. Moodle should automatically create a link to that activity with the autolink class. That link is supposed to be underlined to let user understand it is a link and not a simple plain text. Test also the last minute change: go to the notification page. The box with admin warning about maturity info should now have a green background color.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32321_master
    • Rank:
      39130

      Description

      The autolink class is missing from style/core.css in the "Formal White" theme in Moodle 2.2. This means that links within texts are not properly distinguished as links. They simply look the same as the rest of the text unless you roll over them. The fix is to simply add "a.autolink

      color: Color value is invalid

      to the end of theme/formal_white/style/core.css.

        Activity

        Hide
        Mary Evans added a comment -

        @Daniele
        I have just assigned this 'bug' to you as I thought you would be the best person to fix it.

        Show
        Mary Evans added a comment - @Daniele I have just assigned this 'bug' to you as I thought you would be the best person to fix it.
        Hide
        Daniele Cordella added a comment -

        The fix is trivial but the problem I am facing is different.
        I can not find any occurrence of anchor with class='autolink' browsing moodle.
        Can anyone help me to check the classes that I am adding?
        Thanks in advance.

        Show
        Daniele Cordella added a comment - The fix is trivial but the problem I am facing is different. I can not find any occurrence of anchor with class='autolink' browsing moodle. Can anyone help me to check the classes that I am adding? Thanks in advance.
        Hide
        Daniele Cordella added a comment -

        Paul, can you please tell me where did you find links with autolink class?
        Thanks in advance.

        Show
        Daniele Cordella added a comment - Paul, can you please tell me where did you find links with autolink class? Thanks in advance.
        Hide
        Paul Thayer added a comment -

        Daniele, the "autolinks" show up when you type the title of a course resource in a text. I.e. If I have a quiz activity named "First test", if I type "First test" in a summary, Moodle will automatically create a link to that activity with the autolink class.

        Show
        Paul Thayer added a comment - Daniele, the "autolinks" show up when you type the title of a course resource in a text. I.e. If I have a quiz activity named "First test", if I type "First test" in a summary, Moodle will automatically create a link to that activity with the autolink class.
        Hide
        Daniele Cordella added a comment -

        Thanks Paul
        I just added the missing class as of your suggestion.

        Show
        Daniele Cordella added a comment - Thanks Paul I just added the missing class as of your suggestion.
        Hide
        Daniele Cordella added a comment -

        @Mary: Can I ask you, please, to review this issue and MDL-32380?
        As far as I can see, the total needed time should be lower than 10 minutes.
        Thanks in advance.

        Show
        Daniele Cordella added a comment - @Mary: Can I ask you, please, to review this issue and MDL-32380 ? As far as I can see, the total needed time should be lower than 10 minutes. Thanks in advance.
        Hide
        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
        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
        Dan Poltawski added a comment -

        Hi Daniele,

        I have integrated this now. I noticed there was a change related to the admin plugins css. I integrated this anyway because it seemed sensible. Please could you update the testing instructions to incldue this change though.

        Show
        Dan Poltawski added a comment - Hi Daniele, I have integrated this now. I noticed there was a change related to the admin plugins css. I integrated this anyway because it seemed sensible. Please could you update the testing instructions to incldue this change though.
        Hide
        Daniele Cordella added a comment -

        Thanks Dan, I just modified the testing instructions

        Show
        Daniele Cordella added a comment - Thanks Dan, I just modified the testing instructions
        Hide
        Frédéric Massart added a comment -

        Test successful on master

        Show
        Frédéric Massart added a comment - Test successful on master
        Hide
        Eloy Lafuente (stronk7) added a comment -

        U P S T R E A M I Z E D !

        Many thanks for the hard work, closing this as fixed.

        Ciao

        Show
        Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: