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

Organize style rules of AccessSecretStorageDialog to avoid visual bugs #8395

Merged
merged 21 commits into from
Jun 15, 2022
Merged

Organize style rules of AccessSecretStorageDialog to avoid visual bugs #8395

merged 21 commits into from
Jun 15, 2022

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Apr 22, 2022

Based on #8160

For the reasoning of this PR, please see #8160 (comment) and #8160 (comment)

The reason of the link color bug is that it has lacked a proper style inheritance. Therefore, this PR also organizes styles by nesting them in order to prevent a similar visual bug from occurring again.

Currently, without BEM style in effective, this should be one of the effective ways of protecting components from unexpected regressions, as demonstrated by the PR above.

Signed-off-by: Suguru Hirahara [email protected]

type: task


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

@github-actions github-actions bot added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Apr 22, 2022
@luixxiul luixxiul marked this pull request as ready for review May 11, 2022 15:51
@luixxiul luixxiul requested a review from a team as a code owner May 11, 2022 15:51
@MadLittleMods MadLittleMods added the Z-Community-PR Issue is solved by a community member's PR label Jun 1, 2022
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

oh my, apologies for the insanely late review on this. Will look into process changes to fix this from happening again.

@turt2live turt2live merged commit 1b426ce into matrix-org:develop Jun 15, 2022
@luixxiul luixxiul deleted the AccessSecretStorageDialog2 branch June 15, 2022 15:47
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.

3 participants