refactor(demo-smart): redesign UI for improved developer experience#1702
Merged
roberttran-cc merged 1 commit intomasterfrom Apr 2, 2026
Merged
refactor(demo-smart): redesign UI for improved developer experience#1702roberttran-cc merged 1 commit intomasterfrom
roberttran-cc merged 1 commit intomasterfrom
Conversation
roberttran-cc
force-pushed
the
demo-smart/ui-update
branch
from
7a7505a to
0a62037
Compare
Contributor
|
🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/demo-smart/ui-update/index.html. This preview will be deleted once this PR is closed. |
Contributor
pdesoyres-cc
left a comment
pdesoyres-cc
left a comment
There was a problem hiding this comment.
did not look at the code but found:
- lang selection do not resist accross navigation
- context selection do not resist accross navigation
HeleneAmouzou
approved these changes
Mar 30, 2026
Contributor
HeleneAmouzou
left a comment
HeleneAmouzou
left a comment
There was a problem hiding this comment.
LGTM ! 👏
Thanks a lot for handling this, this new UI looks way better and the developer experience is improve for sure ! 🙏
roberttran-cc
force-pushed
the
demo-smart/ui-update
branch
from
0a62037 to
8b18e34
Compare
The demo-smart page had poor UX: flat unorganized list of ~60 links, cryptic context buttons, single-column layout, no language support. Restructured to sidebar navigation with 21 grouped component categories (organized by domain), right-side context panel appearing only when a component is selected. Added EN/FR language switcher, active link highlighting, responsive design (context panel stacks below component at ≤1400px), native title tooltips, accessibility landmarks, and blue-gray color palette with increased base font-size for readability. Also some cleanup in contexts (removed duplicates, clarified names).
roberttran-cc
force-pushed
the
demo-smart/ui-update
branch
from
8b18e34 to
ce2e7ea
Compare
Member
Author
|
Fixed the language being not persistent. For the context selection, it's a bit more tricky:
So I decided to leave it as is for this revamp (goal being not to spend too much time on it). |
Contributor
|
🔎 The preview has been automatically deleted. |
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
4 participants
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.
Summary
langparamTest plan
npm run storybook:devor servedemo-smart/and verify sidebar navigation, language switching, context buttons, and responsive layout