graasp / graasp/graasp-ui

always show parent in breadcrumb navigation

Open
#591 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
4
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Overall, the breadcrumbs functionality in Builder should be improved.

Specially for deep items, e.g.:

![image](https://github.com/graasp/graasp-ui/assets/1707188/6918a394-c6d5-4d0d-abd7-905a9d0baa65)

The ellipsis covers the parent, which is ofter the most useful place to go.

![image](https://github.com/graasp/graasp-ui/assets/1707188/92041479-7166-4cf8-832b-4c85818efa9b)

I think we should:

1. Not show the current item as a link, but just as text, to avoid confusion.
2. Always show the parent (and add the ellipsis before the parent, if needed).

Potentially:

3. Forward navigation is redundant (as you can always click in on the items shown below and could be removed, to reduce confusion and save a bit of horizontal space.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the Builder breadcrumb component and reviewing how it renders deep item paths. Implement the requested navigation behavior: show the parent, place any ellipsis before it, and render the current item as text rather than a link. Treat removing forward navigation as optional, since the issue presents it as a potential follow-up.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
design, frontend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.