Issue Details (XML | Word | Printable)

Key: MDL-17359
Type: Task Task
Status: Open Open
Priority: Minor Minor
Assignee: moodle.com
Reporter: Tim Hunt
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

Review the language files, and fix all the places we are using < and > as arrows in language strings

Created: 22/Nov/08 11:07 AM   Updated: 28/Nov/08 08:29 PM
Return to search
Component/s: Accessibility, Languages
Affects Version/s: 2.0
Fix Version/s: 2.0

Issue Links:
Relates
 

Participants: moodle.com, Olli Savolainen and Tim Hunt
Security Level: None
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
This is bad for screen-reader users.

If I can find an effective regex, and I can be bothered, I might go beyond the quiz.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Tim Hunt added a comment - 25/Nov/08 09:44 PM
We seem to be OK on <. I checked with the regexp

<(?Unable to render embedded object: File (?[pP][hH][pP])(?) not found..*>)

which finds unmatched < that are not part of an open PHP tag. There are about a dozen in the lang files, all of which are HTML tags in help files wrapped over two lines, except for string addrandom1 in quiz.php, which is no longer used anywhere.


Tim Hunt added a comment - 25/Nov/08 10:14 PM
There are bad > characters in

admin.php string importlangreminder
editor.php string cutpastemozilla
enrol_authroize.php strings logindesc,

They should be replaced by $THEME->rarrow, after calling check_theme_arrows(). This is an accessibility thing.

The quiz string Move to >> is no longer used anywhere.

I have fixed the one genuine occurrence in quiz.php.


Tim Hunt added a comment - 25/Nov/08 10:16 PM
Reassigning, to let the people who maintain those bits of the code fix the remaining occurrences.

Olli Savolainen added a comment - 26/Nov/08 10:28 PM - edited
Tim, in revision 1.73 you seem to have added the following in /mod/quiz/editlib.php, around line 831:

$a = new stdClass;
$a->catname = '<strong>' . $category->name . '</strong>';
$a->arrow = $THEME->rarrow;
echo get_string('addnewquestionsqbank','quiz', $a);

This seems to cause the following in function get_string(), breaking the UI:

Catchable fatal error: Object of class stdClass could not be converted to string in /home/olli/workspace/moodle2cvs/lib/moodlelib.php(5435) : eval()'d code on line 1

I am not familiar with this way of using get_string, so I don't know how to fix it.


Tim Hunt added a comment - 26/Nov/08 11:35 PM
Did you CVS-update everything? (In particular, including the language file) It works for me, but I did change the lang string. (Since it is a very recently introduce string, that is OK.)

Olli Savolainen added a comment - 28/Nov/08 08:28 PM
Oh, I guess it was my bad. Sorry for the disturbance.