First pass at error condition comps for file upload UI

[eawoods]

I expect there to be edits, but wanted to use our existing process to share and review.

This is an illustration of the error conditions we talked about: general errors, delete confirmation, and file-specific errors (for files that are accepted with errors and warnings). Original page design comp is included for comparison.

Note in particular:

• changes in info box wording for clarity and more actual helpful info
• changing widths of things - I am playing around with layout and trying to find a nice balance between keeping blocks of text readable and not trapping or wasting space
• did not yet illustrate the wider file list with file size and timestamp as discussed

This entire PR is images, but including here for ease of review:

File specific errors

General errors

Can be blue for info, orange for warning, can stack. Interested in what you think of the different widths for error boxes.

Upload failure

Delete confirmation page

See https://www.invisionapp.com/blog/microcopy-destructive-actions/ for rationale on wording of buttons.

[coveralls]

Pull Request Test Coverage Report for Build 157

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 70.613%

---

Totals
Change from base Build 139:	0.0%
Covered Lines:	495
Relevant Lines:	701

---

💛 - Coveralls

[eawoods]

@DavidLFielding Thank you for the review! Answering your thoughtful questions:

Error message text matching: assume that the UI will handle language changes if needed, or ignore mismatches in design language and actual error codes for now. We can align that later, when it's closer to finished.

File rename error message: I thought about using the phrase "check references" too, and then thought maybe a user might think we were referring to their paper's references. Perhaps an unfounded concern, would defer to admin team's thinking on that particular piece of wording.

File size/date: yep -- will set that up next!

Primary column: yes it does take up too much space. Mostly because of the heading length for the word "Primary"! Any thoughts on that?

Single file deletion: would get confirmed in exactly the same way as for all files. The current illustrated example assumes that someone pressed the "remove all" button; if only a single file were being removed the language would adjust to reflect the single file -- including display of that file's name. We haven't done much with dynamically changing the page title, but certainly could -- to "Delete all files" for the delete-all case, and "Delete file" (singular) for the opposite case.

In the event that we implement the checkboxes for selective removal of multiple files, I'd imagine that the delete screen would change subtly to match (listing files, title as "Delete selected files", etc).