900025 - Add a <noscript> block to webmaker.org

Status: RESOLVED FIXED

(Webmaker Graveyard :: webmaker.org, defect)

Description

[Andrew Hayward [:arhayward]]

Right now, the webmaker.org site is broken, superficially at least, without JavaScript. The usual expectation when coming to a page and being presented with a 'loading' graphic (feline or otherwise) is that something is loading. However, without JavaScript, nothing changes, suggesting to the user that something somewhere is either taking a long time to load, or is not going to load at all.

This is neither the time nor the place to discuss the decision to pull in all content with JavaScript, but I would suggest that at the very least a <noscript> block be included to let people know that nothing is ever going to load, and they they'll have to enable it to get much further.

Comment 1

[Zakeria Hassan]

Hi,
Can someone assign this bug to me.

Thanks

Comment 2

[Zakeria Hassan]

I removed the static text and used get text. I'm sure there will be localization done to this error page. Please let me know if there is anything else.
Awaiting r+
Ali to review...

Comment 3

[Zakeria Hassan]

Attachment: https://github.com/mozilla/webmaker.org/pull/428

Comment 4

[Ali Al Dallal [:alicoding]]

Comment on attachment 817584 [details] [review]
https://github.com/mozilla/webmaker.org/pull/428

r+ with one nit

Comment 5

[Zakeria Hassan]

is the one nit the new line on line 109 of view/layout.html if it is then I've removed it.

Comment 6

[Jon Buckley [:jbuck]]

Comment on attachment 817584 [details] [review]
https://github.com/mozilla/webmaker.org/pull/428

We've changed the layout of webmaker.org so that the front page is actually pretty functional without Javascript now. https://webmaker.org/starter-makes and https://webmaker.org/teach still need JS though.

I'd recommend editing views/gallery.html to add a <noscript> block in place of the animated kitty gif, rather than doing a page redirect. It'll be less intrusive, and more accurate, since many parts of the site work fine without JS

Comment 7

[Zakeria Hassan]

Comment on attachment 817584 [details] [review]
https://github.com/mozilla/webmaker.org/pull/428

@Jon:

Thanks Jon. I've implemented the fix's to the comment you posted. Please see the latest code at the following link.

https://github.com/mozilla/webmaker.org/pull/428/files

Comment 8

[Jon Buckley [:jbuck]]

Comment on attachment 817584 [details] [review]
https://github.com/mozilla/webmaker.org/pull/428

r+ with nits noted in the PR.

This looks good from a code perspective, but I think the design needs some work: https://dl.dropboxusercontent.com/u/4403845/Screenshots/br.png.

Comment 9

[inactivate account for kate]

Comment on attachment 817584 [details] [review]
https://github.com/mozilla/webmaker.org/pull/428

Ideally the loading cat should also be hidden, but otherwise this looks OK to me

Comment 10

[Zakeria Hassan]

Hi,

Can we set this bug to Resolved status since both reviewers have given me a r+.

Thanks,
Zak

Comment 11

[Ali Al Dallal [:alicoding]]

Please rebase and merge your commits and I will be happy to land that for you.

Comment 12

[Zakeria Hassan]

Ok thanks I will rebase.

Comment 13

[Zakeria Hassan]

Rebase is complete please land when you get a chance.

thank you.

Comment 14

[[github robot]]

Commit pushed to master at https://github.com/mozilla/webmaker.org

https://github.com/mozilla/webmaker.org/commit/a9e1b03f774a91a0f52cb2b8a970c823ce38e149
Bug900025: Add a <noscript> block to webmaker.org only required on /gallery