fix(layout): group floating controls and clear the footer - #85
Merged
Merged
Conversation
hakula139
force-pushed
the
fix/compact-floating-controls
branch
from
September 29, 2026 18:20
2f1f4e9 to
769b74e
Compare
hakula139
force-pushed
the
fix/compact-floating-controls
branch
from
September 29, 2026 21:21
769b74e to
0d02a61
Compare
hakula139
added a commit
to hakula139/hakula.xyz-kiln
that referenced
this pull request
Sep 30, 2026
## Summary - Pin merged IgnIt main `790ee79` and regenerate the site stylesheet for the shared floating controls. The comments shortcut remains available on phones, while back-to-top is hidden below 640px. - Use native fragment links for both actions, restore entry and exit fades, exclude hiding controls from keyboard interaction, and reserve their layout slots so visibility changes do not shift either control. CSS keeps the stack above the footer. - Align the score table with Tailwind's `max-md` breakpoint while retaining its screen-only scope. The desktop rule now starts at 768px. ## Verification - `pnpm build`, `pnpm format`, `pnpm lint`, `pnpm spellcheck`, `nix flake check`, and `git diff --check` passed. Repeated stylesheet builds produced identical output. - The site build rendered 75 pages. Chromium checks at 390px, 640px, and 1440px confirmed unchanged control coordinates throughout hiding and reappearance, opacity fades, blocked focus and pointer interaction while inert, and narrow-screen back-to-top hiding. ## Related PR - [Theme implementation](hakula139/IgnIt#85)
hakula139
added a commit
that referenced
this pull request
Sep 30, 2026
) ## Summary - Keep article heading text aligned with body text. Below 1280px, the heading is a section link that turns blue on hover or keyboard focus. At desktop widths, it keeps its normal color and reveals the original `#` marker on hover or focus. - Use one native section link with a desktop-only CSS marker. Remove heading link underlines, preserve the h2 divider, and wrap long headings. Preserve inline formatting and existing authored links, and leave callout and blockquote headings unchanged. - Retain merged floating controls from #85 and native callouts from #86. ## Verification - `pnpm build`, `pnpm format`, `pnpm lint`, `pnpm spellcheck`, `nix flake check`, commit hooks, `git diff --check`, and CSS sync passed for the theme. The consuming site passed the same checks and its minified build rendered and indexed 75 pages after CSS generation. - Managed headless Chromium checked the generated LR2 / BMS and FFT articles at 320px, 390px, 768px, 1279px, 1280px, and 1440px in light and dark themes. Heading text aligned with body text, section links stayed inside the cards, and pages had no horizontal overflow. Default typography, responsive hover / focus colors, no link underlines, desktop marker visibility and containment, focus outlines, and Enter fragment navigation passed. - Long Chinese and unbroken Latin heading probes wrapped within the card. Inline emphasis and code were preserved, authored links remained intact without nested anchors, and repeated initialization did not duplicate heading links. - Browser checks used generated assets with external resources excluded. Local Nix validation covered aarch64-darwin. Other browser engines and Nix systems were not tested locally.
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
1 participant
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.
Summary
Verification
pnpm build,pnpm format,pnpm lint,pnpm spellcheck,nix flake check, andgit diff --checkpassed. Repeated stylesheet builds produced identical output.Related PR