Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: remove some unused SCSS #4257

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

WofWca
Copy link
Collaborator

@WofWca WofWca commented Oct 24, 2024

I searched the codebase and it appears to not have these classes.

The majority of the elements that use the removed classes
were removed in e04604a,
then import-qr-code-dialog was removed, in e2b23d3.

e6f0521 introduced
ephermeral-message__select-option SCSS, but no elements use it.
git log -S"ephemeral-message" shows only this commit.

I searched the codebase and it appears to not have these classes.

The majority of the elements that use the removed classes
were removed in e04604a,
then `import-qr-code-dialog` was removed, in e2b23d3.

e6f0521 introduced
`ephermeral-message__select-option` SCSS, but no elements use it.
`git log -S"ephemeral-message"` shows only this commit.
@WofWca WofWca force-pushed the wofwca/remove-unused-scss-2 branch from c07e5b3 to adf3005 Compare October 24, 2024 17:36
@WofWca WofWca added the refactor This PR or issue is about refactoring label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor This PR or issue is about refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant