Remove the APPS overview page - #21
Merged
Merged
Conversation
Delete the apps.html overview page and the APPS nav item (desktop and mobile). The QuickScan and Herstelkosten sub-pages are kept as standalone, direct-URL pages and added as explicit `build` entries so they still deploy now that their only inbound link (apps.html) is gone. - Remove src/apps.html - Remove APPS nav item from the header partial; drop the dead data-active-nav="apps" selector in _menu.scss - Neutralize active:"apps" on the quickscan, herstelkosten and nationaal-herstelregister pages - Update package.json start/build scripts, sitemap.xml, llms.txt and llms-full.txt to drop apps.html references Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
1 participant
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
Removes the APPS overview page (
apps.html) and its nav entry, per request. The two app sub-pages (QuickScan, Herstelkosten) are kept as standalone, direct-URL pages.Changes
src/apps.html(the overview/index page).src/_partials/header.html): removed theAPPSnav item from both the desktop and mobile menus._menu.scss): dropped the now-dead[data-active-nav="apps"]active-state selector.active:"apps"→active:""onapps-quickscan.html,apps-herstelkosten.htmlandnationaal-herstelregister.html(the highlight key no longer exists).package.json: removedapps.htmlfrom thestartscript; addedapps-quickscan.htmlandapps-herstelkosten.htmlas explicitbuildentries (see note below).apps.htmlfromsitemap.xml,llms.txtandllms-full.txt; fixed the stale "linked under Apps" phrasing inllms.txt.Note on the sub-pages
apps-quickscan.htmlandapps-herstelkosten.htmlwere previously reachable in the production build only via the<a href>link fromapps.html(Parcel follows links fromindex.html). Deletingapps.htmlorphaned them, so they were added as explicitbuildentries to keep them deployable. They now have no inbound links from the live site and are direct-URL only.Verification
pnpm build:apps.htmlis absent fromdist/;apps-quickscan.htmlandapps-herstelkosten.htmlare present; all other pages still build. SCSS compiles cleanly.200.🤖 Generated with Claude Code