#146 Add a message for noscript users
Closed by darknao. Opened by misc.
fedora-web/ misc/websites add_noscript  into  master

Download 146.patch

So I am using noscript, and going on https://getfedora.org/coreos/download, it do not display anything useful. I figured some messages could help to remind people that the website need JS.

I didn't test how it render everything, so maybe it should go somewhere else.

This would need to be wrapped properly in {% trans %} blocks (see other templates for examples of how to do this properly with links).

Also, only the coreos page(s) should require JS (and that team manages/controls those pages themselves), so we might only want this warning on those pages.

rebased onto fb96033b0d05e198d0e24d635419b19d7e5d8963

From a quick look at the current getfedora prage, we also have some JS code there. However, it seems to be loaded at the end of the HTML code, so I am not sure why.

And I think it is better to show the warning in all cases rather than just where we know this is broken. The fact that no one reported before me kinda imply that nobody tested before, and so others pages might be broken as well.

Prefer something like this, which pulls the link out in a way that makes the translation not lose context:

<noscript>
  {% trans trimmed url="https://pagure.io/fedora-web/websites" %}
  Please enable Javascript if you have trouble seeing part of the website. All
  our code is under a free license and is available on
  <a href="{{ url }}">our forge</a>.
  {% endtrans %}
</noscript>

@darknao, @thunderbirdtr Is this PR relevant anymore?

rebased onto aa75a369c613e78cd43681d3ea129776ba5b5410

rebased onto ffafbacd1a2f2dd19a59e41c7bb83e94fceedb1f

Metadata Update from @darknao:
- Request assigned

Merged manually with 08349eb + d78034b

Pull-Request has been closed by darknao

Metadata