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

Remove an obsolete style block related to highlighting from _IRCLayout.pcss #10451

Merged
merged 4 commits into from
Apr 4, 2023
Merged

Conversation

luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Mar 25, 2023

This PR removes an obsolete style block related to highlighting from _IRCLayout.pcss

Initially it has been added with this commit for the implementation of IRC layout on #4531, but the border declarations for highlighting on _EventTile.scss, which the style block on _IRCLayout.pcss has been expected to cancel, were replaced by this commit on #7209 with ones based on box-shadow.

For reference, you could refer the current implementation from here.

type: task

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

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.

…t.pcss

Initially it has been added with this commit 67249e1 for the implementation of IRC layout, but the border declarations for highlighting were removed with this commit 3a2aa9d to replace the 'border-left' declaration with the 'box-shadow' declaration.

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 Mar 25, 2023
@luixxiul luixxiul mentioned this pull request Mar 25, 2023
3 tasks
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 👍

Thank you for your contribution @luixxiul

@weeman1337
Copy link
Contributor

@luixxiul can you push an empty commit to trigger CI?

It would speed things up if you can enable „Allow edit from maintainers“ on your PRs.

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

luixxiul commented Apr 4, 2023

Thanks for the review!

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