Merge PR #2 from XlebyllleK/master

Refactor code
This commit is contained in:
Eshan Roy
2025-01-11 14:05:15 +05:30
committed by GitHub
3 changed files with 25 additions and 13 deletions

View File

@@ -4,10 +4,14 @@ import { SearchBar } from './components/SearchBar';
import { PackageList } from './components/PackageList'; import { PackageList } from './components/PackageList';
import { usePackages } from './hooks/usePackages'; import { usePackages } from './hooks/usePackages';
import {
Repository
} from './types';
export default function App() { export default function App() {
const { packages, loading, error } = usePackages(); const { packages, loading, error } = usePackages();
const [search, setSearch] = useState(''); const [search, setSearch] = useState('');
const [selectedRepository, setSelectedRepository] = useState<'core' | 'extra' | 'all'>('all'); const [selectedRepository, setSelectedRepository] = useState('all');
const [debouncedSearch, setDebouncedSearch] = useState(search); const [debouncedSearch, setDebouncedSearch] = useState(search);
// Debounce search to optimize performance // Debounce search to optimize performance
@@ -34,7 +38,7 @@ export default function App() {
setSearch(value); setSearch(value);
}; };
const handleRepositoryFilterChange = (repo: 'core' | 'extra' | 'all') => { const handleRepositoryFilterChange = (repo: Repository) => {
setSelectedRepository(repo); setSelectedRepository(repo);
}; };

View File

@@ -1,11 +1,15 @@
import { ThemeToggle } from './ThemeToggle'; import { ThemeToggle } from './ThemeToggle';
import { Logo } from './Logo'; import { Logo } from './Logo';
import {
Repository
} from '../types';
interface HeaderProps { interface HeaderProps {
onRepositoryChange: (repo: 'core' | 'extra' | 'all') => void; onRepositoryChange: (repo: Repository) => void;
} }
export function Header({ onRepositoryChange }: HeaderProps) { export function Header({ onRepositoryChange }: HeaderProps): JSX.Element {
return ( return (
<header className="bg-gradient-to-r from-nord-9 to-nord-8 via-nord-10 text-nord-6 shadow-lg"> <header className="bg-gradient-to-r from-nord-9 to-nord-8 via-nord-10 text-nord-6 shadow-lg">
<div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-4"> <div className="max-w-7xl mx-auto px-4 sm:px-6 lg:px-8 py-4">
@@ -17,14 +21,12 @@ export function Header({ onRepositoryChange }: HeaderProps) {
{/* Repository Filter Dropdown */} {/* Repository Filter Dropdown */}
<div> <div>
<select <select onChange={
onChange={(e) => onRepositoryChange(e.target.value as 'core' | 'extra' | 'all')} (e) => onRepositoryChange(e.target.value as Repository)
defaultValue="all" } defaultValue="all" className="bg-nord-5 dark:bg-nord-1 text-black dark:text-white border-2 border-nord-4 dark:border-nord-2 rounded-lg py-2 px-4 focus:ring-2 focus:ring-nord-8">
className="bg-nord-5 dark:bg-nord-1 text-black dark:text-white border-2 border-nord-4 dark:border-nord-2 rounded-lg py-2 px-4 focus:ring-2 focus:ring-nord-8" {Object.values(Repository).map((repository) => (
> <option key={repository} value={repository}>{((repository === Repository.All) ? 'All Repositories' : repository.charAt(0).toUpperCase() + repository.slice(1))}</option>
<option value="all">All Repositories</option> ))}
<option value="core">Core</option>
<option value="extra">Extra</option>
</select> </select>
</div> </div>

View File

@@ -1,5 +1,11 @@
// Type alias for repository categories // Type alias for repository categories
export type Repository = 'core' | 'extra' | 'community' | 'multilib'; export enum Repository {
All = 'all',
Core = 'core',
Extra = 'extra',
Community = 'community',
Multilib = 'multilib'
}
// Interface representing a single package // Interface representing a single package
export interface Package { export interface Package {