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

Remove unique settings from labs user settings tab #8828

Merged
merged 5 commits into from
Jun 14, 2022
Merged

Remove unique settings from labs user settings tab #8828

merged 5 commits into from
Jun 14, 2022

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Jun 13, 2022

This PR removes the unique settings from _LabsUserSettingsTab.scss in order to make the spacing same as the other tabs, other than the setting which specifies the toggle switch position.

Before After
before after

type: task

Signed-off-by: Suguru Hirahara luixxiul@users.noreply.github.com


This change is marked as an internal change (Task), so will not be included in the changelog.

Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
@github-actions github-actions bot added Z-Community-PR Issue is solved by a community member's PR T-Task Refactoring, enabling or disabling functionality, other engineering tasks labels Jun 13, 2022
@luixxiul luixxiul marked this pull request as ready for review June 13, 2022 05:42
@luixxiul luixxiul requested a review from a team as a code owner June 13, 2022 05:42
Copy link
Contributor

@weeman1337 weeman1337 left a comment

Choose a reason for hiding this comment

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

Hi @luixxiul thank you for your contribution!

Can you check my comment and re-request review after resolving it?

Signed-off-by: Suguru Hirahara <luixxiul@users.noreply.github.com>
Copy link
Contributor

@weeman1337 weeman1337 left a comment

Choose a reason for hiding this comment

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

LGTM now 👍

@weeman1337 weeman1337 merged commit 2c751a4 into matrix-org:develop Jun 14, 2022
@luixxiul
Copy link
Contributor Author

I am actually not really satisfied with the naming --SettingsTab_section-margin-bottom-preferences-labs but the variable is used only in those special cases so it looks fine now. Needs normalization.

@luixxiul luixxiul deleted the LabsUserSettingsTab branch June 14, 2022 08:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants