Page MenuHomePhabricator

Remove uses of .box-shadow() mixin in Growth team extensions
Closed, ResolvedPublic

Description

The .box-shadow() LESS mixin is now obsolete as box-shadow is supported without any vendor prefix fallbacks, and planned to be removed by the next MediaWiki release. Affected Growth team extensions per codesearch: GrowthExperiments, Echo, Flow

Event Timeline

kostajh edited projects, added Growth-Team (Current Sprint); removed Growth-Team.

Change 713883 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/GrowthExperiments@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713883

Change 713884 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/Echo@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713884

Change 713885 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/Flow@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713885

Change 713883 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713883

Change 713884 merged by jenkins-bot:

[mediawiki/extensions/Echo@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713884

Change 713885 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] Remove uses of .box-shadow() mixin

https://gerrit.wikimedia.org/r/713885

Tgr added a subscriber: Etonkovidova.

This is done (thanks @kostajh!).

IMO it's too trivial to check and can just be resolved but will leave it to @Etonkovidova to decide.

Checked on betalabs - a quick regression testing for GrowthExperiments, Echo, and Flow features. All look as expected.