egullickson/motovaultpro

fix: Add document modal file input bottom padding (#19) #20

Merged

egullickson merged 4 commits from issue-19-document-modal-padding into main

2026-01-05 00:52:47 +00:00

Author	SHA1	Message	Date
Eric GullicksonandClaude Opus 4.5	0e9d94dafa	fix: Wrap file input in flex container for vertical centering (refs #19 ) Deploy to Staging / Build Images (pull_request) Successful in 2m42s Details Deploy to Staging / Deploy to Staging (pull_request) Successful in 38s Details Deploy to Staging / Verify Staging (pull_request) Successful in 6s Details Deploy to Staging / Notify Staging Ready (pull_request) Successful in 5s Details Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped Details File inputs are replaced elements that ignore CSS centering properties. The only reliable solution is to wrap the input in a flex container with items-center. Changes: - Added wrapper div with `flex items-center h-11` - Moved border/background/focus styles to the wrapper - Input now uses flex-1 to fill available space - Used focus-within for focus ring on wrapper 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>	2026-01-04 18:45:25 -06:00
Eric GullicksonandClaude Opus 4.5	75d1a421d4	fix: Use line-height for file input vertical centering (refs #19 ) Deploy to Staging / Build Images (pull_request) Successful in 2m41s Details Deploy to Staging / Deploy to Staging (pull_request) Successful in 37s Details Deploy to Staging / Verify Staging (pull_request) Successful in 6s Details Deploy to Staging / Notify Staging Ready (pull_request) Successful in 5s Details Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped Details Use leading-[44px] to match the h-11 height, which should vertically center the file input content. Removed padding that was conflicting. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>	2026-01-04 18:36:45 -06:00
Eric GullicksonandClaude Opus 4.5	1534f33232	fix: Vertically center file input content (refs #19 ) Deploy to Staging / Build Images (pull_request) Successful in 2m39s Details Deploy to Staging / Deploy to Staging (pull_request) Successful in 37s Details Deploy to Staging / Verify Staging (pull_request) Successful in 6s Details Deploy to Staging / Notify Staging Ready (pull_request) Successful in 5s Details Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped Details The "Choose File" button and "No file chosen" text were not vertically centered within the file input box. Fixed by: - Using py-2.5 for input padding (10px top/bottom) - Adding file:my-auto to center the button vertically - Adjusting file:py-1.5 for button internal padding Note: flex/items-center don't work on <input> elements as they are replaced elements. Using padding and margin-auto instead. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>	2026-01-04 18:28:09 -06:00
Eric GullicksonandClaude Opus 4.5	510420e4fd	fix: Vertically center file input content (refs #19 ) Deploy to Staging / Build Images (pull_request) Successful in 2m39s Details Deploy to Staging / Deploy to Staging (pull_request) Successful in 37s Details Deploy to Staging / Verify Staging (pull_request) Successful in 5s Details Deploy to Staging / Notify Staging Ready (pull_request) Successful in 5s Details Deploy to Staging / Notify Staging Failure (pull_request) Has been skipped Details The "Choose File" button and "No file chosen" text were not vertically centered within the file input. This was caused by: 1. Browser default `align-items: baseline` for file inputs 2. Conflicting `py-2` padding on the input container Fixed by: - Removing `py-2` (conflicting vertical padding) - Adding `flex items-center` (explicit vertical centering) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <[email protected]>	2026-01-04 18:21:14 -06:00