Edgewall Software

Opened 7 years ago

Closed 7 years ago

Last modified 4 years ago

#12874 closed defect (fixed)

Button to switch interactive/plain-text view doesn't work if preferred language is non-English — at Version 5

Reported by: Jun Omae Owned by: Jun Omae
Priority: normal Milestone: 1.3.2
Component: i18n Version: 1.3dev
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Fixed switch to plain-text/interactive mode in error.html when preferred language is not English.

Description

Because this.value.indexOf("interactive") < 0 doesn't work as expected if preferred language is non-English, at trunk/trac/templates/error.html@15964:39,40,45#L38:

Line 
38 $("#tbtoggle").click(function() {
39 if (this.value.indexOf("interactive") < 0) {
40 this.value = j_("Switch to interactive view");
41 $("#traceback ul").fadeOut("fast", function() {
42 $("#traceback pre").fadeIn("fast");
43 });
44 } else {
45 this.value = j_("Switch to plain text view");
46 $("#traceback pre").fadeOut("fast", function() {
47 $("#traceback ul").fadeIn("fast");
48 });
49 }
50 return false;
51 });

Change History (5)

comment:1 by Jun Omae, 7 years ago

Owner: set to Jun Omae
Status: newassigned

Proposed changes in [9186ecf40/jomae.git] (jomae.git@t12874).

comment:2 by Jun Omae, 7 years ago

Release Notes: modified (diff)
Summary: Button to switch intractive/plain-text view doesn't work if preferred language is non-EnglishButton to switch interactive/plain-text view doesn't work if preferred language is non-English

comment:3 by Ryan J Ollos, 7 years ago

Changes look good to me.

comment:4 by Jun Omae, 7 years ago

Resolution: fixed
Status: assignedclosed

Thanks for the viewing. Committed in [16155].

comment:5 by Ryan J Ollos, 4 years ago

Internal Changes: modified (diff)
Release Notes: modified (diff)
Note: See TracTickets for help on using tickets.