look: fix scrolling to sections #34

Closed
q3k wants to merge 4 commits from q3k-fix-link-targeting into main
Owner

This fixes:

  1. The inability to scroll down to 'contacts' and other lower sections.
    We just expand the second-to-last section to allow the browser to
    scroll.
  2. Updating the currently active section - this now uses a global view
    of all visible sections that is updated by the observer diff, thereby
    fixing the logic.
This fixes: 1. The inability to scroll down to 'contacts' and other lower sections. We just expand the second-to-last section to allow the browser to scroll. 2. Updating the currently active section - this now uses a global view of all visible sections that is updated by the observer diff, thereby fixing the logic.
q3k added 1 commit 2025-04-13 17:03:32 +00:00
look: fix scrolling to sections
Some checks failed
/ deploy (push) Failing after 2s
8c52de0616
This fixes:

1. Sections overlapping with navbar when linked to. The fix is dirty,
   it's just a margin.
2. The inability to scroll down to 'contacts' and other lower sections.
   We just expand the second-to-last section to allow the browser to
   scroll.
3. Updating the currently active section - this now uses a global view
   of all visible sections that is updated by the observer diff, thereby
   fixing the logic.
q3k force-pushed q3k-fix-link-targeting from 8c52de0616 to 06ff6d32e8 2025-04-13 17:06:35 +00:00 Compare
palid added 2 commits 2025-04-13 17:58:58 +00:00
palid added 1 commit 2025-04-13 18:49:28 +00:00
fix: make it work with worse motions
Some checks failed
/ deploy (push) Failing after 1s
fb3d783ebc
Collaborator

Solved by #35

Solved by #35
palid closed this pull request 2025-04-13 19:41:29 +00:00
Some checks failed
/ deploy (push) Failing after 1s

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: infra/site#34
No description provided.