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

Conversation

@luixxiul
Copy link
Contributor

@luixxiul luixxiul commented May 14, 2023

This is a follow-up to bef8e07#diff-5d3e25eb1d22dd7a96e0e87ea9c1298dc7dbce8eb965d30b2d1ebc1436bfec3fR840.

This PR intends to prevent a visual regression by extracting style rules specific to ThreadsList and not applying them to Notification. Obviously mx_ThreadPanel_replies is not required for EventTile on NotificationPanel, and the rules should be updated not to be applied to it.

1

type: task

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

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

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

@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 May 14, 2023
@luixxiul luixxiul marked this pull request as ready for review May 19, 2023 07:03
@luixxiul luixxiul requested a review from a team as a code owner May 19, 2023 07:03
@luixxiul luixxiul requested review from dbkr and florianduros May 19, 2023 07:03
@dbkr dbkr added this pull request to the merge queue May 22, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 22, 2023
@germain-gg germain-gg added this pull request to the merge queue Jun 1, 2023
Merged via the queue into matrix-org:develop with commit 610ee53 Jun 1, 2023
@luixxiul luixxiul deleted the EventTile branch June 1, 2023 15:31
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