Skip to content

fix(pat-upload): keep buttons inline and hide empty queue controls - #1668

Open
MrTango wants to merge 1 commit into
masterfrom
pr/upload-buttons-layout
Open

MrTango wants to merge 1 commit into
masterfrom
pr/upload-buttons-layout

Conversation

@MrTango

@MrTango MrTango commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Keep upload buttons beside their fields at narrow widths. Respect the template's initially hidden queue controls and reduce excess preview spacing.

 upload rows
-  col-md-9 + col-md-3
+  flexible col + content-sized col-auto
 queue controls
-  force display: block
+  preserve initially hidden state

Evidence

  • Before (styles/template): Columns stack below the medium breakpoint; SCSS overrides the hidden controls state.
    After: Rows use breakpoint-independent columns and no longer force controls visible.
  • pnpm test --runInBand: 59 suites passed; 512 tests passed, 15 skipped.
  • pnpm build:webpack: passed with size/performance warnings.
  • No manual responsive/browser verification in this pass.

Merge Danger

Door: two-way

Blast Radius: layout

Affects all pat-upload instances, including content-browser upload forms.

Assisted-by: GPT 6.1 Sol

…ntrols

- the Browse and Upload buttons take their intrinsic width, col-md-3 was
  too narrow for them in a small column and broke their labels
- .upload-queue .controls no longer forces display: block, which
  out-specified the display: none that _showHideControls starts from, so
  the Upload button and progress bar showed with an empty queue
- reduce the 4em gap above the previews to 1em
@MrTango
MrTango marked this pull request as ready for review November 13, 2026 08:07
@MrTango
MrTango requested a review from petschki November 13, 2026 08:07

This branch has not been deployed

No deployments
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

Successfully merging this pull request may close these issues.

1 participant