zyratlo opened a new pull request, #88:
URL: https://github.com/apache/incubator-texera-site/pull/88

   ### What changes were proposed in this PR?
   Adds a top offset to blog post pages, whose titles are currently hidden 
behind the fixed navbar at viewports of 768px and up.
   - The site reserves space for the navbar with `.td-default 
main>section:first-of-type { padding-top: 4.5rem !important; }` in 
`assets/scss/_styles_project.scss`. That selector only matches pages whose 
`main` has a direct `<section>` child. `layouts/blog/baseof.html` renders a 
post into `main > div.td-content` with no `<section>` anywhere, so the rule 
never applies, and the `h1` sits at `top: 0` beneath the 66px `position: fixed` 
navbar.
   - Other layouts avoid this by hardcoding the offset inline 
(`layouts/_default/single.html` uses `padding-top: 6rem`, 
`layouts/releases/single.html` uses `9rem`). The blog layout never had one.
   - The new rule is scoped to `body.td-blog main>.td-content` and sits inside 
the same `min-width: 768px` media query as the existing rule, because below 
that breakpoint the navbar is `position: relative` and never overlaps the title.
   
   ### Before / After
   <img width="1512" height="836" alt="Screenshot 2026-09-15 at 11 29 57 AM" 
src="https://github.com/user-attachments/assets/38d96b6f-9339-4e5b-b2ff-92ad7357806b";
 />
   <img width="1512" height="836" alt="Screenshot 2026-09-15 at 11 33 46 AM" 
src="https://github.com/user-attachments/assets/309e24d2-3c52-4f71-a10b-92a8bc987473";
 />
   
   
   ### Any related issues, documentation, discussions?
   Not tied to one post. All existing posts are affected, and every future one 
would be. Noticed while preparing a new blog post.
   
   ### How was this PR tested?
   - Checked all existing posts and confirmed each title clears the navbar
   - Confirmed nothing else moves: the blog index (`/blog/`), `/docs/`, and the 
homepage have no direct `main > .td-content`, so the selector does not match 
them
   - Narrowed the viewport below 768px, confirmed no change, since the navbar 
is in normal flow there and never overlapped
   
   ### Was this PR authored or co-authored using generative AI tooling?
   Co-authored with Claude Opus 5 in compliance with ASF


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to