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.

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

Breadcrumbs should be scrollable #8714

Closed
swedneck opened this issue Feb 15, 2019 · 0 comments
Closed

Breadcrumbs should be scrollable #8714

swedneck opened this issue Feb 15, 2019 · 0 comments

Comments

@swedneck
Copy link

No description provided.

@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
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants