Depends on: 1421424
Priority: P2 → P1
Bryan, here's a place to put or link to the mocks for error handling boxes...
Here's a mock for an error state for the sections. The box styling should be the same that we use for zero-states, although, note that there's no icon in the box as not to draw too much attention. If the area needs to be shorter (i.e. Top Sites section) it can be the appropriate size for that section and the text should float in the middle. Copy reads: "Oops, something went wrong loading this content. Refresh the page to try again." The last line should be a link that reloads the page. https://mozilla.invisionapp.com/share/YFFA2STTB#/272847648_New_Tab_-_Section_Error
The pull request: https://github.com/mozilla/activity-stream/pull/3964
Commit pushed to master at https://github.com/mozilla/activity-stream https://github.com/mozilla/activity-stream/commit/5e46ed45e4c021561503a2c1dd709dd60129932f Merge pull request #3964 from dmose/2-react-16 Upgrade to React 16.2 and add error boundaries (fix bug 1421441)
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Strings landed for localization in https://github.com/mozilla/activity-stream-l10n/pull/11
Whiteboard: [strings needed][AS60MVP] → [strings landed][AS60MVP]
You need to log in before you can comment on or make changes to this bug.