Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Add spacing between right panel and main timeline#7176

Merged
germain-gg merged 2 commits into
developfrom
gsouquet/autohidescrollbar-padding-19795
Nov 22, 2021
Merged

Add spacing between right panel and main timeline#7176
germain-gg merged 2 commits into
developfrom
gsouquet/autohidescrollbar-padding-19795

Conversation

@germain-gg
Copy link
Copy Markdown
Contributor

@germain-gg germain-gg commented Nov 22, 2021

Fixes element-hq/element-web#19795
Regressed by #7098

Port to release PR: #7176

@toger5 to flesh out a long term fix for the maximised widget project


This PR currently has no changelog labels, so will not be included in changelogs.

Add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is plus X-Breaking-Change if it's a breaking change.

Preview: https://619b927054e726d25fa68587--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

@germain-gg germain-gg requested a review from a team as a code owner November 22, 2021 12:06
Copy link
Copy Markdown
Contributor

@toger5 toger5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this.
I think increasing the left margin of the Apps drawer by 8px (Left maring: (5+8)px ). would not hurt to get the apps drawer symmetrical again.

current state:
image

Copy link
Copy Markdown
Contributor

@toger5 toger5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@germain-gg germain-gg merged commit 9e7b407 into develop Nov 22, 2021
@germain-gg germain-gg deleted the gsouquet/autohidescrollbar-padding-19795 branch November 22, 2021 13:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

We seem to have lost timeline scrollbar padding

3 participants