Replace the timetable filter pills with one Filters button and popover
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 42/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- react, typescript
Piste de recherche
Start with ViewTimeTable.tsx, FacultySelector.tsx, and the Figma frame “Timetable filters” (node 2560:1371); check the open work on #266, #283, and #262 before editing. Build the Filters button, popover, active filter tags, and clear/remove interactions while keeping faculty and same-building behavior. Done when the design and listed interactions work; coordinate the remaining timetable logic with #289.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Scroll down for design
Is your feature request related to a problem? Please describe.
Next to "Your Timetables" there are Morning, Evening and Mixed pills, a Same Building checkbox and a Filter by Faculty box. They take a lot of room and each one is a separate control.
Describe the solution you'd like
One Filters button replaces all of them. Pressing it opens a popover.
- A count badge on the button shows how many filters are on.
- Popover options:
- Theory time (Morning, Evening, Mixed)
- Class time: a range bar with two handles that snap to class start and end times, with the times shown under the handles
- Fewest lab days
- Same building
- A faculty field
- Clear all
- Active filters show as small removable tags beside "Your Timetables", and the × removes one.
- Same building and the faculty field keep working as they do today, just moved into the popover. How the other options pick and order timetables is in the logic issue (#289).
Additional context
Related: #187 (filter by multiple faculties), #188 (filter by slots) and #263 (same-building filtering), all closed.
ViewTimeTable.tsx is changing in other work, so expect merge conflicts:
- #266 (open): splits
FacultySelector.tsxandViewTimeTable.tsxinto smaller modules. - #283 (open PR): changes timetable state in
ViewTimeTable.tsx. - #262 (open): changes the saved-timetable local storage syncing in the same file.
- #267 (merged): groups timetables with the same slot layout. The tags and counts must still refer to the real timetable selection.
Design
- Figma frame:
Timetable filters(node2560:1371) on the page "FFCS revamp". It contains theFilters button, theFilters popoverand theActive filtersrow. - Link: https://www.figma.com/design/uCP8B7drfprKgsHYjB8sYs?node-id=2560-1371
- The Figma frame still shows the old Fewest 8 am / Fewest 7:20 pm options; the range bar replaces them.
Example
One way to build it (just an idea):
ZButtonfor the Filters button- A popover built like the other popups
- The existing faculty
ComboBoxfor the faculty field
<ZButton type="regular" text={`Filters${count ? ` (${count})` : ''}`} onClick={togglePopover} />
{open && <FiltersPopover value={filters} onChange={setFilters} onClear={clearAll} />}
- Langage dominant
- TypeScript
- Étoiles
- 6
- Forks
- 37
- Merge moyen
- 11 h 13 min
- PR mergées (30 j)
- 55
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de CodeChefVIT/ffcs
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
CodeChefVIT/ffcs#295 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 1-2 jours Accessibilité débutants 48/100
CodeChefVIT/ffcs#292 ·
Les mainteneurs répondent en général sous 1 jour
-
Edit a course from Your Courses with a pencil buttonPeut-être pris @Aiden-Trindade l’a pris il y a 2 jours. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 43/100
CodeChefVIT/ffcs#291 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Put the selection box and Your Courses side by side and restyle themPeut-être pris @Aiden-Trindade l’a pris il y a 2 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
CodeChefVIT/ffcs#290 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 43/100
CodeChefVIT/ffcs#289 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de CodeChefVIT/ffcs
Issues similaires
-
Replace discontinued crypto-js in the themePeut-être pris @aelmanaa l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
PaloAltoNetworks/docusaurus-openapi-docs#1689 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
solana-foundation/solana-web3.js#4015 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
formatSearchQuery drops the :present modifier, so `email:present=true` is sent as `email=true`Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 90/100
ivopogace/opencode-courier#422 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
chapel-lang/chapel-vscode#158 ·
Les mainteneurs répondent en général sous 1 jour