Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review and Address issues from Front End Checklist #562

Closed
28 of 42 tasks
murny opened this issue Mar 14, 2018 · 0 comments
Closed
28 of 42 tasks

Review and Address issues from Front End Checklist #562

murny opened this issue Mar 14, 2018 · 0 comments
Assignees

Comments

@murny
Copy link
Contributor

murny commented Mar 14, 2018

Full list found here: https://github.com/thedaviddias/Front-End-Checklist

Broken down list (for what I think is relevant?) is below:

HEAD

  • Put important meta tags at top of page (charset, viewport, etc)
  • Truncate long title tags, google wants them to be around 55 characters
  • Generate Favicons for different devices/browsers/etc
  • rel="noopener"
  • Canonical url
  • Facebook OG and Twitter Cards
  • Need to figure out meta description (with regards to work already being done here Crawler Friendliness & Hardening #483 and we need the copy for the main description)

HTML

  • Semantic elements being used
  • Linting of HTML, W3C compliant, broken links etc.

CSS

  • Website is responsive
  • Website tested on various devices: various combinations of desktop, mobile, browsers, operating systems
  • Unique Ids are actually unique on the page
  • js prefix on css classes
  • no embedded or inline css

Images

  • Images have been optimized for web (thanks imgbot?)
  • width/height set on <img> tag
  • alt text being used properly
  • Should use a better logo, that has better dimensions

Javascript

  • No inline javascript
  • Noscript tag?
  • Do we need jquery everywhere?
  • ESlint?

Security

Performance

  • First meaningful paint under 1 sec, time to interactive under 5 secs for average configuration (crappy android phone on slow 3g network) and 2 secs for repeat visits
    • Big suggestion is to use a CDN for javascript/css assets (cookieless domain/cachable/etc)
    • Add headers like " Vary: Accept-Encoding Header", "Last-Modified or ETag " etc.
  • Google pagespeed testing, scoring at least 90/100

Accessibility

  • Progressive enhancement (major stuff should work without javascript)
    • Everything seems to work except Deposit (as it does a post request using javascript so this 404s)
  • Color contrast should pass WCAG AAA
  • All pages have an H1 and right order for sub headings (H1 to H6)
  • Semantic HTML5 input types are used
  • Labels are used (aria-label if label can't be displayed)
  • Wave testing done
  • Keyboard navigation testing done
  • Screen reader testing?
  • Focus style?

SEO

  • Google analytics
  • Structured data
  • robots.txt
  • Sitemap.xml
  • Verify pagination links have proper rel="prev/next"
  • HTML sitemap?
@murny murny self-assigned this Mar 20, 2018
murny added a commit that referenced this issue Mar 27, 2018
@murny murny mentioned this issue Apr 5, 2018
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants