Fixed Wrong case for JsOnly class in member_view

Chris D

XenForo developer
Staff member
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.webp

Vs, this:

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

Still looks awful. But then that serves you right if you want to browse a modern site with JS turned off :p
 
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.
 
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".
 
Back
Top Bottom