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

Add spacing between right panel and main timeline #7176

Merged
merged 2 commits into from
Nov 22, 2021

Conversation

germain-gg
Copy link
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
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
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