Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Gradient on the edge of breadcrumbs is unclickable #8890

Closed
turt2live opened this issue Feb 22, 2019 · 0 comments
Closed

Gradient on the edge of breadcrumbs is unclickable #8890

turt2live opened this issue Feb 22, 2019 · 0 comments
Assignees
@turt2live
Labels
A-Breadcrumbs P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Milestone
Breadcrumbs: Out ...

Comments

@turt2live
Copy link
Member

image

I can't click the room after the B, but would very much like to do so
@bwindels bwindels mentioned this issue Mar 1, 2019
Closed
@lampholder lampholder added T-Defect ui/ux P2 S-Minor Impairs non-critical functionality or suitable workarounds exist A-Breadcrumbs labels Mar 4, 2019
@turt2live turt2live added this to the Breadcrumbs: Out of labs milestone Mar 27, 2019
@turt2live turt2live self-assigned this Mar 27, 2019
turt2live added a commit to matrix-org/matrix-react-sdk that referenced this issue Mar 27, 2019
@turt2live
Support horizontal scrolling on breadcrumbs
9b64dd0 
Fixes element-hq/element-web#8714
Fixes element-hq/element-web#8890
Fixes element-hq/element-web#9034
Fixes element-hq/element-web#8954

This turned out to be much more complicated than it needed to be. We use an IndicatorScrollbar to do all the math for us and some minor changes have been made so it can flag left/right overflow. The complicated part is the css changes which make the gradients work: unlike the RoomSubList, we have to calculate the offset of the indicators (gradients) on our own because position:sticky doesn't work horizontally.

The changes to the css (well, mostly pointer-events:none) make it so the gradient doesn't interfere with the room avatars. 

9034 and 8954 are fixed by this because they represent an overflow-x:none style breakage where browsers won't let you scroll without a scrollbar. The gradient offset problem is also demonstrated in 8954.
@turt2live turt2live mentioned this issue Mar 27, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@turt2live turt2live

Labels
A-Breadcrumbs P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Milestone
Breadcrumbs: Out of labs
Development

No branches or pull requests
2 participants
@turt2live @lampholder