You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As we do not have a concrete policy or a tool to prevent style rules from becoming obsolete (creating a tracker or linter to detect obsolete rules is indeed unrealistic. see: #25149), I am confident that nobody really knows how much style rules have become obsolete.
Since the rules removed by those PRs have been found just coincidentally, we should be able to detect a lot of obsolete rules if we check CSS files for them intentionally.
How would you like to achieve it?
There is not a tool for automation, therefore, manual labor; investigate each file, remove obsolete rules, create E2E tests if necessary.
Have you considered any alternatives?
I imagine that introducing Compress should reduce a chance of letting rules become obsolete, but it does not detect rules which have already become obsolete, unless it would replace the whole UI, making all of the rules obsolete all at once (which I believe would not happen).
Additional context
No response
For references, here is a list of all CSS files under res/css:
Thanks for reporting this. Given that this will need to be an ongoing effort, I think we should first explore automated tooling for finding dead CSS. Therefore, I'm closing this in favor of #23067.
Your use case
What would you like to do?
Investigate CSS files for obsolete style rules and remove them.
Why would you like to do it?
With recent PRs (matrix-org/matrix-react-sdk#10733, matrix-org/matrix-react-sdk#10687, matrix-org/matrix-react-sdk#10659, matrix-org/matrix-react-sdk#10550, matrix-org/matrix-react-sdk#10548, matrix-org/matrix-react-sdk#10496, and matrix-org/matrix-react-sdk#10451), I have removed CSS style rules which have been obsolete since several years.
As we do not have a concrete policy or a tool to prevent style rules from becoming obsolete (creating a tracker or linter to detect obsolete rules is indeed unrealistic. see: #25149), I am confident that nobody really knows how much style rules have become obsolete.
Since the rules removed by those PRs have been found just coincidentally, we should be able to detect a lot of obsolete rules if we check CSS files for them intentionally.
How would you like to achieve it?
There is not a tool for automation, therefore, manual labor; investigate each file, remove obsolete rules, create E2E tests if necessary.
Have you considered any alternatives?
I imagine that introducing Compress should reduce a chance of letting rules become obsolete, but it does not detect rules which have already become obsolete, unless it would replace the whole UI, making all of the rules obsolete all at once (which I believe would not happen).
Additional context
No response
For references, here is a list of all CSS files under
res/css
:_AuthButtons.pcss
,_NewSessionReviewDialog.pcss
, and_ManageIntegsButton.pcss
matrix-org/matrix-react-sdk#10753_AuthButtons.pcss
,_NewSessionReviewDialog.pcss
, and_ManageIntegsButton.pcss
matrix-org/matrix-react-sdk#10753_AuthButtons.pcss
,_NewSessionReviewDialog.pcss
, and_ManageIntegsButton.pcss
matrix-org/matrix-react-sdk#10753The text was updated successfully, but these errors were encountered: