diff options
| author | Adam Mathes <adam@adammathes.com> | 2026-02-16 10:53:59 -0800 |
|---|---|---|
| committer | Adam Mathes <adam@trenchant.org> | 2026-02-16 10:57:38 -0800 |
| commit | 5cf8275540d7162cd4936a7c0e76dbfe7f66b62c (patch) | |
| tree | fdb3c47560f1b1556f7c203f72d50d13f171c95a /frontend-vanilla/src/main.ts | |
| parent | 96e78c5fdfada73d37644083c7580a1d444ed748 (diff) | |
| download | neko-5cf8275540d7162cd4936a7c0e76dbfe7f66b62c.tar.gz neko-5cf8275540d7162cd4936a7c0e76dbfe7f66b62c.tar.bz2 neko-5cf8275540d7162cd4936a7c0e76dbfe7f66b62c.zip | |
V3 UI Polish: Improved keyboard navigation, fixed logo position, and updated branding
- Fix V3 keyboard navigation delay (resolved NK-wjats7)
- Update V3 document title to 'neko' (resolved NK-4p3s91)
- Fix V3 neko logo/button position to be top-left fixed (resolved NK-89za3s)
- Improve FeedItems (React) stability with ref-based index tracking and robust tests
- Sync V3 styling and selection feedback with V2 patterns
- Rebuild production assets
Diffstat (limited to 'frontend-vanilla/src/main.ts')
| -rw-r--r-- | frontend-vanilla/src/main.ts | 43 |
1 files changed, 30 insertions, 13 deletions
diff --git a/frontend-vanilla/src/main.ts b/frontend-vanilla/src/main.ts index 3f53fc2..7b55c48 100644 --- a/frontend-vanilla/src/main.ts +++ b/frontend-vanilla/src/main.ts @@ -185,6 +185,14 @@ export function attachLayoutListeners() { const itemRow = target.closest('.feed-item'); if (itemRow && !itemTitle) { // Clicking the row itself (but not the link) const id = parseInt(itemRow.getAttribute('data-id')!); + activeItemId = id; + + // Update visual selection + document.querySelectorAll('.feed-item').forEach(el => { + const itemId = parseInt(el.getAttribute('data-id') || '0'); + el.classList.toggle('selected', itemId === activeItemId); + }); + const item = store.items.find(i => i._id === id); if (item && !item.read) { updateItem(id, { read: true }); @@ -252,7 +260,7 @@ export function renderItems() { contentArea.innerHTML = ` <ul class="item-list"> - ${items.map((item: Item) => createFeedItem(item)).join('')} + ${items.map((item: Item) => createFeedItem(item, item._id === activeItemId)).join('')} </ul> ${store.hasMore ? '<div id="load-more-sentinel" class="loading-more">Loading more...</div>' : ''} `; @@ -682,19 +690,28 @@ window.addEventListener('keydown', (e) => { function navigateItems(direction: number) { if (store.items.length === 0) return; - let index = store.items.findIndex(i => i._id === activeItemId); - index += direction; - if (index >= 0 && index < store.items.length) { - activeItemId = store.items[index]._id; - const el = document.querySelector(`.feed-item[data-id="${activeItemId}"]`); - if (el) el.scrollIntoView({ block: 'nearest' }); - // Optional: mark as read when keyboard navigating - if (!store.items[index].read) updateItem(activeItemId, { read: true }); - // Since we are in 2-pane, we just scroll to it. - } else if (index === -1) { - activeItemId = store.items[0]._id; + const currentIndex = store.items.findIndex(i => i._id === activeItemId); + let nextIndex; + + if (currentIndex === -1) { + nextIndex = direction > 0 ? 0 : store.items.length - 1; + } else { + nextIndex = currentIndex + direction; + } + + if (nextIndex >= 0 && nextIndex < store.items.length) { + activeItemId = store.items[nextIndex]._id; + + // Update visual selection without full re-render for speed + document.querySelectorAll('.feed-item').forEach(el => { + const id = parseInt(el.getAttribute('data-id') || '0'); + el.classList.toggle('selected', id === activeItemId); + }); + const el = document.querySelector(`.feed-item[data-id="${activeItemId}"]`); - if (el) el.scrollIntoView({ block: 'nearest' }); + if (el) el.scrollIntoView({ block: 'start', behavior: 'smooth' }); + + if (!store.items[nextIndex].read) updateItem(activeItemId, { read: true }); } } |
