Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Quiz
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      questionsperpageselected and shufflequestionsselected from lang/en_utf8/quiz.php need to use a $a, rather than concatenation.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            pilpi Olli Savolainen added a comment -

            Hm, I added a string which contained >> instead of >>,

            $string['addnewquestionsqbank'] = 'Add questions to the category $a in the \'Question bank contents\' tool >>';

            but as it seems there was one already

            $string['moveto'] = 'Move to >>';

            I started wondering if you guys actually have a justification for this?

            Show
            pilpi Olli Savolainen added a comment - Hm, I added a string which contained >> instead of >>, $string ['addnewquestionsqbank'] = 'Add questions to the category $a in the \'Question bank contents\' tool >>'; but as it seems there was one already $string ['moveto'] = 'Move to >>'; I started wondering if you guys actually have a justification for this?
            Hide
            timhunt Tim Hunt added a comment -

            On the subject of >>, actually that is bad for accessibility. (Screen readers read that out "greater-than, greater-than, which is not good.) The correct thing to do is:

            1. First make sure that you have called check_theme_arrows();
            2. Then use $THEME->larrow and $THEME->rarrow with $a. Instead of < and >

            I think also that check_theme_arrows does the right thing when the user is using a RTL language.

            You can see an example in http://cvs.moodle.org/moodle/admin/roles/assign.php?view=markup, starting from the comment '/// Print the form'. Oh dear, bad example, it is another place where we concatenate language strings that has not been fixed.

            But since there are other place that get this wrong, you don't have to fix it yourself. I'll create a new bug, assigned to me, to fix all these before the Moodle 2.0 release.

            Show
            timhunt Tim Hunt added a comment - On the subject of >>, actually that is bad for accessibility. (Screen readers read that out "greater-than, greater-than, which is not good.) The correct thing to do is: 1. First make sure that you have called check_theme_arrows(); 2. Then use $THEME->larrow and $THEME->rarrow with $a. Instead of < and > I think also that check_theme_arrows does the right thing when the user is using a RTL language. You can see an example in http://cvs.moodle.org/moodle/admin/roles/assign.php?view=markup , starting from the comment '/// Print the form'. Oh dear, bad example, it is another place where we concatenate language strings that has not been fixed. But since there are other place that get this wrong, you don't have to fix it yourself. I'll create a new bug, assigned to me, to fix all these before the Moodle 2.0 release.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10