fix(special): align most linked pages list layout - #1963
Closed
andriypolanski wants to merge 2 commits into
Closed
Conversation
|
Verdict: Close PR #1963: fix(special): align most linked pages list layout This PR changes UI, layout, styling, responsive behavior, or interaction behavior, but the PR description does not clearly include before/after author-provided screenshots or a short screen recording. Visual PRs need evidence in the PR description so the maintainer can review the change quickly and reliably. A deploy preview link alone is not enough. Required evidence for resubmission:
Changed files:
CI: No visible GitHub checks yet. Please open a new focused PR with the required evidence attached. Automated review by Taopedia Site Maintainer. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Align the Most linked pages special page list layout so article titles and link counts are easier to scan.
Changes in
src/pages/wiki/special/mostlinkedpages.astro:<ol>to<ul>(unordered list, no numbered markers).padding-left: 32px→padding: 0).margin-left: autoon.mw-ml-count).Affected route:
/wiki/special/mostlinkedpages/This is a website-only UI change in this repository. No article/content changes from
taopedia-articlesare included, consistent with the repo split described in README.md.Why This Should Ship
The Most linked pages page is a navigation surface for the most-referenced articles across Taopedia. The previous layout indented the list away from the page title and did not clearly separate article names from link counts. The updated layout matches the intended reading pattern: article name on the left, link count on the right, aligned with the page heading.
Ranking data is unchanged. The page still uses
buildMostLinkedPages()fromscripts/most-linked.js, the same source as/wiki/special/mostlinkedpages.json.Review Notes
Per CONTRIBUTING.md:
test(notmain).--border-color-subtle,--color-base-subtle); no hardcoded light/dark colors./wiki/special/mostlinkedpages/Most linked pagesheadingValidation
npm run dev # Open http://localhost:4321/wiki/special/mostlinkedpages/ npm run build npm run test:most-linkedExisting regression checks in
scripts/check-most-linked.jsstill match the same row markup (mw-ml-row,mw-ml-title,mw-ml-count); only list semantics and CSS changed.Target Branch
testmainonly because it promotestestto productionSecurity Checklist
Test Plan
/wiki/special/mostlinkedpages/locally and confirm the list is an unordered list (no numbers).Most linked pagesheading.npm run test:most-linkedafternpm run build.