SUBMISSION-150: Update Review Files page with v5-styling.#69
Merged
Conversation
…to a friendly HTML page) when the bucket has no PDF (edgecase); compilation is curently broken so self-heal when PDF copied into place (review): when the bucket has a PDF but submission.preview is None or stale, fire ConfirmSourceProcessed and ConfirmPreview atomically so the second event sees the in-memory state from the first. The legacy DB doesn't persist submission.preview, so a reload between events would lose it. INFO logging for each branch. [SUBMISSION-122] David
…harden /preview.pdf: catch NotFound from the controller and 302 to /preview_not_available (non-.pdf URL so the browser renders HTML instead of invoking the PDF viewer on a .pdf URL that returned HTML), and make set_etag/Content-Length best-effort so missing blob metadata doesn't crash the response. [SUBMISSION-122] David
…exists in bucket) and pass to template so an absent PDF can never produce an enabled Submit button. Skip FinalizeSubmission unless the form action is next -- the nav bar's Go Back and Save & Exit buttons also POST this form, which would accidentally submit when the proceed checkbox happens to be ticked. INFO logging on render. [SUBMISSION-122] David
…it gated on preview_ready (preview viewed AND PDF still exists in bucket); banner branches three ways (no PDF / unviewed PDF / ready); sidebar adds Download Submission Agreement link and editing-after-submitting copy; new shortcuts row with Preview PDF (tracking preview state), Edit Metadata, Edit Files, Edit License; JS state machine reloads the page on tab return after a PDF view; amber highlight on the proceed checkbox row when ticking it is the last remaining gate. [SUBMISSION-122] David
…nd Approve' -> 'Confirm and Submit'. Sets the pagetitle context var which arxiv-base renders into <title> in the page head. [SUBMISSION-122] David
…ew" -> "Confirm", title "Final preview" -> "Confirm and Submit". Class name stays so the endpoint URL is unchanged; docstring notes the rename. [SUBMISSION-122] David
…ubmit_nav and expose it via named blocks so per-page templates can customize the Submit button without rewriting the macro: submit_nav gains next_label/next_aria/next_disabled/next_id (backward-compat defaults of "Continue" / "Save and continue" / not disabled / no id preserve current behavior for every existing page), and base_edit.html wraps the top/bottom submit_nav calls in submit_nav_top and submit_nav_bottom blocks so final_preview can override and render "Submit" instead. [SUBMISSION-122] David
…rsor on disabled .submit-nav buttons because Bulma's default opacity-only disabled styling reads as active on a saturated green Submit, and add .proceed-highlight (soft amber row + bolder label + brief pulse animation) to draw the eye to the proceed checkbox when ticking it is the only remaining gate before Submit. [SUBMISSION-122] David
…directs here because the bucket has no PDF: explains the situation as unexpected (the PDF should exist by Confirm step), suggests retry/reprocess/contact-support, and offers Back-to-Confirm / Go-to-Process / Close-tab buttons. Lives at a non-.pdf URL so the browser renders HTML instead of invoking its built-in PDF viewer. [SUBMISSION-122] David
…mission Agreement link, plus the reportlab dependency it needs. Generates a personalized PDF stamped with the submission's ID, submitter, submission date, license, and agreement_id using reportlab Platypus; body text is placeholder pending canonical legal copy from arXiv. [SUBMISSION-122] David
…lob metadata on read so crc32c, size, and updated timestamps are populated, avoiding ETag/Content-Length failures and giving callers real metadata. [SUBMISSION-122] David
…greement controller commit. [SUBMISSION-122] David
… submission type warning (moved to Review Files step). [SUBMISSION-149] David
…ilds a fresh .tar.gz on each request from the current source files in the bucket. Linked from a new Download Package button in the Upload Files sidebar; same button to follow on the Review Files page. [SUBMISSION-150] David
…ng forward Submit 1.5's sidebar content (Did you know..., Accepted Formats, Important, Read more about v1.5) in the new heading hierarchy. Adds two action buttons missing from the v5 Upload mockup but useful on Review: Download Package and Back to upload step (links to ui.file_upload). [SUBMISSION-150] David
…ad of 500 or stack trace. We now catch error and display a warning without the empty selections and file list that require the preflight report. Suggest retry or contact help. [SUBMISSION-150] David
… the Submit 1.5 page layout. Added current sidebar content from 1.5 as placeholder for review. Some additional CSS to support new review cards. [SUBMISSION-150] David
|
How does this handle multiple top level files? |
|
Are we using Red Text in form headings in other places? |
|
Will figure size be displayed? |
bdc34
reviewed
Jun 3, 2026
bdc34
left a comment
bdc34
left a comment
Contributor
There was a problem hiding this comment.
Some changes/questions to answer.
Contributor
|
Thank you for the screenshots, David. The metadata page mockup can be used a style reference for form fields, field laberls, fieldsets, etc. Quick screenshot: Once I can review the page(s) in a browser I can do an internal usability, accessibility, and style review. And schedule in test user sessions. For now the styles on other pages can be referenced, and I think the metadata page will be the closest guide. |
…-style layout: commit ca59dbd accidentally emptied submit_overrides.css, removing the functional disabled-button and proceed-checkbox-highlight styles needed by the Confirm page along with the sidebar polish. Restored to the pre-wipe 18f6c38 state;eliminated red headings carried over from 1.5. [SUBMISSION-150] David
…orm in a single .action-container matching add_metadata.html. [SUBMISSION-150] David
Contributor
Author
This does not appear to work. I have created https://arxiv-org.atlassian.net/browse/SUBMISSION-170. |
Contributor
Author
The red headings have been eliminated with the latest updates. I updated the screenshot in the description. |
Contributor
Author
The size dooes not appear to be getting displayed in 2.0. We will need to review the 1.5 updates we added to display size and flag figures that may cause issues with compilation. 1.5: I added a ticket: https://arxiv-org.atlassian.net/browse/SUBMISSION-172 |
bdc34
approved these changes
Jun 9, 2026
…tore.py (_get_checksum: use develop's get_blob structure with SUBMISSION-150's None-guard), process.py/preview.py (port SUBMISSION-150's enhanced file_preview into develop's new preview.py module), routes/ui.py (use renamed cntrls.new.preview.file_preview call site, keep SUBMISSION-150's preview_not_available + submission_agreement_pdf routes and ETag/size defensive guards), review.py (combine Markup import with develop's SetDecisions/SetDirectivesAndCleanup imports), upload.py (drop SUBMISSION-150's source-format workaround now that SUBMISSION-128 populates submission.source_format at upload time), test_review.py (keep develop's expanded test suite, apply SUBMISSION-150's assertion-text reword, fix _get_csrf helper to mock preflight as populated so the form renders), file_upload.html (restructure as info_container_middle / more_notifications / within_content sibling blocks). David
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There is currently no v5 mockup for the Review Files page. This PR attempts to bring the existing Submit 2.0 page closer to the 1.5 look and feel while adopting the v5 styling currently used in Submit 2.0.
For reference, I will include the 1.5, original 2.0, and updated 2.0 Review Files screenshots.
@bmaltzan I rearranged the main content section and added content to sidebar based on the Submit 1.5 Review Files page. The ordering of the selections changed slightly. If this was changed by subsequent design decisions or looks off please let me know.
@SBBCornell @jimentwood I added you to PR so you are aware of the changes. I've included screenshots to allow for a quick look at the changes. There will be a more thorough UAT at a future date.
Submit 1.5 Review Files
Submit 2.0 Review Files (original)
Submit 2.0 Review Files (updated per 1.5 and v5 style)
Submit 2.0 Review Files - Preflight call failure
When preflight request fails (rare event) the UI displayed a stack trace. The page relies heavily on preflight so page content ends up blank so we hide the selections and file list.