bvandeusen/FabledCurator
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases 35 Wiki Activity

fix(ui): remove redundant app-shell top gap (#1480) #225

Merged
bvandeusen merged 1 commits from dev into main 2026-07-13 14:12:41 -04:00
Conversation 0 Commits 1 Files Changed 1
+6 -6
bvandeusen commented 2026-07-13 14:12:32 -04:00
Owner
Copy Link
Copy Source

Follow-up to the batch-4 live review. Removes the double-offset that put a large empty band at the top of every view: .fc-content padded 64px to clear the navbar, but TopNav is position: sticky and already reserves its own space — the 64px was a fixed-navbar leftover. Content now flows directly under the sticky nav (and the full-height Explore/Subscriptions views no longer overflow). Frontend-only, no migration. CI-green on dev 447bf73.

🤖 Generated with Claude Code

Follow-up to the batch-4 live review. Removes the double-offset that put a large empty band at the top of every view: `.fc-content` padded 64px to clear the navbar, but `TopNav` is `position: sticky` and already reserves its own space — the 64px was a fixed-navbar leftover. Content now flows directly under the sticky nav (and the full-height Explore/Subscriptions views no longer overflow). Frontend-only, no migration. CI-green on `dev` `447bf73`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
bvandeusen added 1 commit 2026-07-13 14:12:32 -04:00
fix(ui): remove redundant app-shell top gap (sticky nav double-offset)
CI / lint (push) Successful in 3s
Details
CI / frontend-build (push) Successful in 22s
Details
CI / backend-lint-and-test (push) Successful in 33s
Details
CI / integration (push) Successful in 3m44s
Details
447bf73519 
Every view showed a large empty band at the top (operator-flagged during the
Vuetify-4 review; pre-existing). Cause: .fc-content had padding-top:64px to clear
the navbar, but TopNav is position:sticky and already reserves its own space in
the v-app flex column — the 64px was a fixed-navbar leftover that double-counted
the offset. Removed it; content now flows directly below the sticky nav (and the
full-height calc(100vh-64px) views no longer overflow).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
bvandeusen merged commit 7d1c701b67 into main 2026-07-13 14:12:41 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
renovate-bot bvandeusen
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bvandeusen/FabledCurator#225
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?