• This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn more.

Fixed Wrong case for JsOnly class in member_view

Chris D

XenForo developer
Staff member
#1
HTML:
<span class="jsOnly">{xen:phrase loading}...</span>
Should be:

HTML:
<span class="JsOnly">{xen:phrase loading}...</span>
The adverse effect is this:

upload_2013-12-4_15-25-51.png

Vs, this:

upload_2013-12-4_15-27-34.png

Still looks awful. But then that serves you right if you want to browse a modern site with JS turned off :p
 

Mike

XenForo developer
Staff member
#2
These mistakes were actually in a number of templates - mostly relating to ignored indicators. We did actually fix this already, but only in 1.3; I believe because of how minor it potentially is and the number of templates it does affect.

However, the reasonable other approach is to support .jsOnly as well.
 

Chris D

XenForo developer
Staff member
#3
Thanks Mike.

On a slightly related note... There isn't a great deal of context with regards to these View links that are visible when <noscript> so might it be worth changing them to "View Recent Activity" or similar? Or at least just use the same phrase name from the tab heading itself rather than just "View".