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

Include mx_MessageActionBar_maskButton in mx_MessageActionBar #8792

Merged
dbkr merged 5 commits into
matrix-org:developfrom
luixxiul:MessageActionBar
Jun 16, 2022
Merged

Include mx_MessageActionBar_maskButton in mx_MessageActionBar #8792
dbkr merged 5 commits into
matrix-org:developfrom
luixxiul:MessageActionBar

Conversation

@luixxiul
Copy link
Copy Markdown
Contributor

@luixxiul luixxiul commented Jun 8, 2022

Based on #8780

Since mx_MessageActionBar_maskButton is not (and should not be) used outside of mx_MessageActionBar, it should be enforced code-wise in order to protect the style rules from being stolen from anywhere else.

Even if the same kind of buttons are to be used somewhere else, style rules should be implemented there, not by stealing the rules implemented for message action bar.

after1
after2

On pinned message card:

after

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

type: task


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

luixxiul added 2 commits June 8, 2022 23:46
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 8, 2022
@luixxiul luixxiul marked this pull request as ready for review June 11, 2022 08:40
@luixxiul luixxiul requested a review from a team as a code owner June 11, 2022 08:40
@dbkr dbkr merged commit 3b28b11 into matrix-org:develop Jun 16, 2022
@luixxiul luixxiul deleted the MessageActionBar branch June 16, 2022 14:37
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