Skip to content

fix(skills): focus the newest skill on the skills page - #211

Merged
Tomlord1122 merged 1 commit into
mainfrom
dev
Oct 21, 2026
Merged

Tomlord1122 merged 1 commit into
mainfrom
dev

Conversation

@Tomlord1122

Copy link
Copy Markdown
Member

Summary

  • The skills page sorts recent changes to the top, but a fresh visit still opened the alphabetically first skill from the API list.
  • Select the top visible skill instead, and scroll the selected row into view.
  • Keep an explicit ?skill= request and refresh-with-keep-selection behavior unchanged.

Test plan

  • pnpm test (196 files, 1268 tests)
  • Open the skills page after a recent skill change and confirm the top recent skill is selected, not the alphabetical first row

The recent-first list still opened the alphabetically first skill, so a fresh visit landed on the wrong row. Select the top visible skill and scroll it into view.
@Tomlord1122
Tomlord1122 merged commit d4c2fac into main Oct 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant