Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Update deprecation-cop.less #18266

Merged
merged 2 commits into from
Nov 1, 2018
Merged

Update deprecation-cop.less #18266

merged 2 commits into from
Nov 1, 2018

Conversation

RyanPisuena
Copy link
Contributor

Requirements for Contributing a Performance Improvement

Description of the Change

Removing '!important argument from background-color of deprecation-cop'

Quantitative Performance Benefits

According to #18174 , removing !important makes changing background color changes easier

Possible Drawbacks

May mess with current background color settings

Verification Process

Did not verify

Applicable Issues

N/A

Delete !important argument from background-color of depreciation-op
Copy link
Contributor

@simurai simurai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 on removing !important.

Maybe !important could also be removed when hovering the .deprecation-info?

background-color: @background-color-selected !important;

Copy link
Contributor

@simurai simurai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 on removing !important.

Maybe !important could also be removed when hovering the .deprecation-info?

background-color: @background-color-selected !important;

Delete !important from hover
@RyanPisuena
Copy link
Contributor Author

@simurai changes made

@simurai
Copy link
Contributor

simurai commented Nov 1, 2018

CI failure seems unrelated (timeout), merging.

Thanks @RyanPisuena 🙇

@simurai simurai merged commit 110c28f into atom:master Nov 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants