feat: modern UI redesign with categories for govt websites - #10
Open
thenishantsapkota wants to merge 2 commits into
Open
feat: modern UI redesign with categories for govt websites#10thenishantsapkota wants to merge 2 commits into
thenishantsapkota wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
bsantosh909
previously approved these changes
Jul 29, 2026
There was a problem hiding this comment.
Pull request overview
This PR modernizes the site’s UI with a new typographic system, reusable UI utility classes, and a redesigned navigation/home experience, while adding category-based filtering to the Government Websites page.
Changes:
- Introduces new Tailwind theme tokens (fonts, shadows, animation) and global component classes (
card,search-input,btn-*). - Redesigns Home page, navbar/footer, and layout spacing to match the updated visual system.
- Updates Tech Companies and Government Websites pages with a refreshed search UI; adds category tabs + filtering for government websites.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tailwind.config.js | Adds font families, new color shade, shadows, and a fade-in animation/keyframes. |
| src/pages/tech-companies.tsx | Updates the listing UI to use new shared components and a redesigned search experience. |
| src/pages/homepage.tsx | Replaces placeholder intro with a new hero + feature-card landing layout. |
| src/pages/government-websites.tsx | Adds categorization + category tabs and refreshes the card/search UI. |
| src/layouts/default.layout.tsx | Adjusts layout structure and container sizing for the new design. |
| src/index.css | Adds base typography + reusable UI component utility classes. |
| src/components/app/navbar.tsx | Redesigns navbar (sticky + active link styling) for the new UI. |
| src/components/app/footer.tsx | Updates footer styling to match the new layout system. |
| index.html | Loads Google Fonts (Inter + Space Grotesk) used by the new Tailwind font families. |
Comments suppressed due to low confidence (4)
src/pages/tech-companies.tsx:88
- The tech company logo
is missing an
altattribute, which hurts screen reader accessibility. Add a meaningful alt text (oralt=""if you intentionally want it to be decorative).
<img
src={`https://raw.githubusercontent.com/Know-Nepal/tech-companies/main/logos/${company.logoName}`}
width={96}
height={96}
src/pages/government-websites.tsx:67
allCategoriesis recomputed on every render even though it only depends ondata. Memoizing it avoids extra work when typing in the search box or switching tabs.
const allCategories = [
...new Set(allItems.map((item) => getCategory(item.name))),
].sort();
src/pages/government-websites.tsx:73
new Fuse(allItems, …)runs on every render. For larger lists this can become noticeable when the user types; memoize the Fuse instance so it's only rebuilt whenallItemschanges.
const fuse = new Fuse(allItems, {
keys: ["name"],
threshold: 0.3,
ignoreLocation: true,
});
src/pages/government-websites.tsx:144
- This image is missing an
altattribute. If it's decorative (since the name/description are already shown), setalt=""so screen readers skip it.
<img
src="/nepalLogo.png"
width={80}
height={80}
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+68
to
+71
| value={searchTerm} | ||
| onChange={(e) => setSearchTerm(e.target.value)} | ||
| placeholder="Search for tech company" | ||
| className="search-input pl-11" |
Comment on lines
+107
to
+112
| type="text" | ||
| placeholder="Search for government website..." | ||
| value={searchQuery} | ||
| onChange={(e) => setSearchQuery(e.target.value)} | ||
| className="search-input pl-11" | ||
| /> |
- Add aria-label to search inputs on both list pages - Add alt text to logo images - Memoize Fuse instance and category list with useMemo
bsantosh909
approved these changes
Jul 29, 2026
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.
No description provided.