[PR #176] [MERGED] fix(link): Keybind, scroll behaviour, restrict drag to vertical #96

Closed
opened 2025-12-29 18:28:19 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/linsa-io/linsa/pull/176
Author: @Aslam97
Created: 9/21/2024
Status: Merged
Merged: 9/21/2024
Merged by: @Aslam97

Base: mainHead: fix/link


📝 Commits (8)

  • 0fb122a chore: expose scrollActiveElementIntoView
  • 0811ba1 feat(utils): editable element
  • 2f427d8 fix: memoize exceptionRefs, use animation frame and check editable element
  • 99f8df6 fix: improve btn on mobile
  • 1197026 chore(drps): bump framer motion version
  • 63c2d94 fix(link): big fix
  • 9319a2a chore: remove comment code
  • b5f70f4 feat: touch device

📊 Changes

14 files changed (+450 additions, -383 deletions)

View changed files

web/app/custom.css (+11 -0)
📝 web/app/globals.css (+1 -0)
📝 web/components/routes/link/LinkRoute.tsx (+6 -41)
📝 web/components/routes/link/bottom-bar.tsx (+23 -19)
📝 web/components/routes/link/header.tsx (+2 -2)
📝 web/components/routes/link/hooks/use-link-actions.ts (+7 -5)
📝 web/components/routes/link/list.tsx (+148 -121)
📝 web/components/routes/link/partials/link-item.tsx (+83 -66)
📝 web/components/routes/page/list.tsx (+1 -1)
📝 web/components/routes/topics/list.tsx (+1 -1)
📝 web/hooks/use-active-item-scroll.ts (+3 -2)
web/hooks/use-touch-sensor.ts (+20 -0)
📝 web/lib/utils/index.ts (+18 -0)
📝 web/package.json (+126 -125)

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/linsa-io/linsa/pull/176 **Author:** [@Aslam97](https://github.com/Aslam97) **Created:** 9/21/2024 **Status:** ✅ Merged **Merged:** 9/21/2024 **Merged by:** [@Aslam97](https://github.com/Aslam97) **Base:** `main` ← **Head:** `fix/link` --- ### 📝 Commits (8) - [`0fb122a`](https://github.com/linsa-io/linsa/commit/0fb122acdde3bd32868cb2dc74a5401c9b34cfaf) chore: expose scrollActiveElementIntoView - [`0811ba1`](https://github.com/linsa-io/linsa/commit/0811ba1ed3608acd1fdeeece60dde5243bc813ba) feat(utils): editable element - [`2f427d8`](https://github.com/linsa-io/linsa/commit/2f427d8bd3c6ee160854718c0b2e51742fcc4f01) fix: memoize exceptionRefs, use animation frame and check editable element - [`99f8df6`](https://github.com/linsa-io/linsa/commit/99f8df61a372e1eb0649c10b0faea457ebc197bf) fix: improve btn on mobile - [`1197026`](https://github.com/linsa-io/linsa/commit/1197026b23622aa6d23e692b732eeb9c0ca8e89d) chore(drps): bump framer motion version - [`63c2d94`](https://github.com/linsa-io/linsa/commit/63c2d94eea8ea7a626757967e354ff8e9a1c3b9c) fix(link): big fix - [`9319a2a`](https://github.com/linsa-io/linsa/commit/9319a2a933175194cabb20d6e3c89a63fa983b1c) chore: remove comment code - [`b5f70f4`](https://github.com/linsa-io/linsa/commit/b5f70f43d8c04f0480de5b6d73507db76b877733) feat: touch device ### 📊 Changes **14 files changed** (+450 additions, -383 deletions) <details> <summary>View changed files</summary> ➕ `web/app/custom.css` (+11 -0) 📝 `web/app/globals.css` (+1 -0) 📝 `web/components/routes/link/LinkRoute.tsx` (+6 -41) 📝 `web/components/routes/link/bottom-bar.tsx` (+23 -19) 📝 `web/components/routes/link/header.tsx` (+2 -2) 📝 `web/components/routes/link/hooks/use-link-actions.ts` (+7 -5) 📝 `web/components/routes/link/list.tsx` (+148 -121) 📝 `web/components/routes/link/partials/link-item.tsx` (+83 -66) 📝 `web/components/routes/page/list.tsx` (+1 -1) 📝 `web/components/routes/topics/list.tsx` (+1 -1) 📝 `web/hooks/use-active-item-scroll.ts` (+3 -2) ➕ `web/hooks/use-touch-sensor.ts` (+20 -0) 📝 `web/lib/utils/index.ts` (+18 -0) 📝 `web/package.json` (+126 -125) </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 18:28:19 +01:00
adam closed this issue 2025-12-29 18:28:19 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/linsa#96