UI/UX first: visual README, GitHub Pages portal, fix all dead nav links#44
UI/UX first: visual README, GitHub Pages portal, fix all dead nav links#44
Conversation
…nav links - Replace placeholder README.md with visual landing page (clickable badges, portal links, org directory, plain-language copy) - Add .github/workflows/pages.yml to deploy blackroad-pixel/ to GitHub Pages - Fix all 6 dead href='#' nav links in blackroad-pixel/index.html (Dashboard, Repositories, Files, Docs, Tools, Agents) - Fix app.js nav handler that blocked real link navigation via e.preventDefault() - Add visual hero section to profile/README.md with launch buttons and portal quick-links Co-authored-by: blackboxprogramming <118287761+blackboxprogramming@users.noreply.github.com>
Workflow Changes DetectedThis PR modifies GitHub Actions workflows. Please ensure:
Automated workflow review |
There was a problem hiding this comment.
Pull request overview
This PR improves first-run UI/UX by turning the repo into a “click-to-use” entrypoint: a visual README/org profile, a GitHub Pages deployment for the pixel portal, and functional navigation links inside the portal.
Changes:
- Rewrote
README.mdinto a visual landing page with CTAs and a quick-link table to the portal and games. - Added a GitHub Actions workflow to deploy
blackroad-pixel/to GitHub Pages on pushes tomain. - Replaced dead
href="#"top-nav links in the pixel portal and adjusted the JS click handler to only prevent default on#links.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
README.md |
New visual landing page + direct links to portal/games and repo map. |
.github/workflows/pages.yml |
Adds Pages deployment pipeline for blackroad-pixel/. |
blackroad-pixel/index.html |
Replaces placeholder nav links with real destinations. |
blackroad-pixel/app.js |
Allows real links to navigate by not always calling preventDefault(). |
profile/README.md |
Updates org profile README with branded hero + launch buttons. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| [](https://blackroad-os.github.io/.github/) | ||
| | ||
| [](https://blackroad-os.github.io/.github/games/) | ||
| | ||
| [](https://blackroad-os.github.io) |
There was a problem hiding this comment.
The “Website” CTA badge text says blackroad.io, but the link currently points to https://blackroad-os.github.io. This is misleading and inconsistent with other repo docs that treat https://blackroad.io as the canonical website; update the href to the intended domain, or change the badge label to match the GitHub Pages URL.
| const href = item.getAttribute('href'); | ||
| // Only prevent default for same-page anchors; let real links through | ||
| if (!href || href === '#') { | ||
| e.preventDefault(); | ||
| } | ||
| navItems.forEach(n => n.classList.remove('active')); | ||
| item.classList.add('active'); | ||
| }); |
There was a problem hiding this comment.
This click handler always toggles the .active class even when the user clicks an external link (e.g., Repositories/Docs) that opens in a new tab. This can leave the current page highlighting an item that didn’t actually navigate; consider only updating the active state for in-portal navigation (relative URLs / same-page anchors) and skipping it for http(s):// links.
| [](https://blackroad-os.github.io/.github/) | ||
| | ||
| [](https://blackroad-os.github.io/.github/games/) | ||
| | ||
| [](https://blackroad-os.github.io) | ||
|
|
There was a problem hiding this comment.
The org profile “Website” button currently links to https://blackroad-os.github.io (GitHub Pages) rather than the main site used elsewhere (https://blackroad.io). If the intent is to send visitors to the primary website, update the href; otherwise rename the button to clarify it’s a GitHub Pages site.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| concurrency: | ||
| group: pages | ||
| cancel-in-progress: false |
There was a problem hiding this comment.
concurrency.cancel-in-progress is set to false, which can allow an older Pages deploy run to finish after a newer one and overwrite the site with stale content. Set cancel-in-progress: true (or use a commit-specific group) so the latest push reliably wins.
| const href = item.getAttribute('href'); | ||
| // Only prevent default for same-page anchors; let real links through | ||
| if (!href || href === '#') { | ||
| e.preventDefault(); | ||
| } |
There was a problem hiding this comment.
The comment says "Only prevent default for same-page anchors", but the condition only matches '#' (placeholder) and will not cover real same-page anchors like #section. Either adjust the comment to reflect the actual intent (placeholder # links) or broaden the condition to handle all hash anchors if that’s what you want.
The repo had a placeholder README ("Enter file contents here"), six non-functional
href="#"nav links in the pixel portal, and no hosted URL — making the entire UI inaccessible without cloning and running locally.Changes
README.md— full rewrite.github/workflows/pages.yml— new GitHub Pages deploymentblackroad-pixel/to GitHub Pages on push tomainhttps://blackroad-os.github.io/.github/; games at.../games/blackroad-pixel/index.html— nav linkshref="#"; replaced with real destinations:blackroad-pixel/app.js— nav click handlere.preventDefault()unconditionally, silently swallowing all nav clicks; now only prevents default on#anchorsprofile/README.md— org profile heroOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.