Repository navigation
Conversation
- panel open state, expanded sections and source categories live in the filters store so they survive Chat <-> Search switches - header: title/chevron as a button, status line with count + clear all always visible; pills hidden on phones when collapsed - filters panel above results on mobile Search (ColumnTemplate filtersFirst prop) - clear-search as a Bulma delete in the text box corner Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nslated error - clear chat moved to a toolbar above the messages and confirmed via ModalComponent - scroll-to-bottom is a sticky, labelled button with one style at all widths - mobile bubbles keep the avatar beside the text; markdown lists indented; left-aligned text - error message translated; mobile 'go to sources' only when sources exist Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…footer contact - at 1330px the logo switches to logo-symbol.png, nav labels become tooltips and the nav top space goes, together - TooltipComponent gets an isBelow option, used by the nav - nav icons normalised, phone row fits iPhone SE, language menu opens leftwards - remove the scroll script that hid the header via direct DOM edits - footer shows 'Contact' (mailto) so it fits one line on phones in FR Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- dvh so the app fits the visible screen on phones - Likert scale fits phone width with legends at the edges - narrower, centred tutor status modal - terms page readable width and title wording Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
noor-lpi
force-pushed
the
fix/ui-filters-chat-nav
branch
from
October 8, 2026 08:57
d669358 to
9a4260f
Compare
| <a | ||
| href="mailto:welearn@learningplanetinstitute.org" | ||
| title="welearn@learningplanetinstitute.org" | ||
| >{{ $t('contact') }}</a |
Collaborator
There was a problem hiding this comment.
se changement est deja fait sur main
sandragjacinto
requested changes
Oct 9, 2026
sandragjacinto
left a comment
Collaborator
There was a problem hiding this comment.
avec l'arrivée du login il y a un nouveau boutton dans la nav. en mode téléphone on ne le voit plus, est-ce que tu penses que tu peux faire le fix dans cette PR ? merci !
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.
Mostly UI polish across Chat, Search and the header. Easiest to review commit by commit: each commit is one area, and no line is touched by more than one commit.
Filters & search
filtersstore and resets on reload. On narrow screens the panel starts collapsed.filtersFirstprop onColumnTemplate).delete) in the corner of the text box, away from the search button.Chat
ModalComponent.<button>with an aria-label, pinned to the bottom of the chat, with the same style at every width.Header, nav & footer
logo-symbol.png, cut fromlogo.png), an icon-only nav with the labels as tooltips, and no extra top space. The logo stays level with the icons.TooltipComponentgets anisBelowoption, used by the nav. Screen readers still read each label once.mailto:link, with the address on hover), so it fits on one line on phones in French.Layout & misc
dvhunits so the app fits the visible screen on phones and the nav no longer scrolls away.Test plan
Known issue (pre-existing, not fixed here)
🤖 Co-generated with Claude Code