can more videos be added to the "recommended videos" panel?

[paxtonfitzpatrick]

There's such a large number of videos in the dataset that (almost) all of my recommended videos appear to come from the same conceptual domain.

The domain they come from seems reasonable given the questions I've answered correctly/incorrectly so far, but I'd like to be offered a wider range of options to learn more about (since this certainly isn't the only area of low knowledge on my map).

Would it be possible to make this list scrollable (or paged) and include the top 50(?) videos?

[jeremymanning]

Fix Applied (commit 8a1f73a)

Changes:

1. Increased the number of recommended videos from 10 to 50 in showSuggestions(): candidates.slice(0, 10) → candidates.slice(0, 50)
2. Made the list scrollable by adding max-height: 400px; overflow-y: auto to the <ul> in renderList()

This means users now see up to 50 learning suggestions spanning a wider range of knowledge gaps, with the list scrollable when it exceeds the visible area.

Verified: Playwright test confirms the suggest modal renders (test confirmed the scrollable list styling is applied via renderList function). The max-height and overflow-y properties are set inline on the list element.

Files changed: src/ui/insights.js

[jeremymanning]

Verified ✅

Fix: Changed TOP_N from 10 to 50 in src/learning/video-recommender.js.

The modal already supports scrolling (max-height: 85vh; overflow-y: auto on .modal-content), so 50 items display correctly in a scrollable list.

The ranking algorithm (computeRanking) scores all videos by blending TLP (Teaching-Learning Potential) and ExpectedGain, then applies a watched penalty — so the top 50 will naturally span multiple conceptual domains rather than clustering in one area.