Skip to content
Permalink
Browse files

Fix AssetAdmin icon reference

Needs to use namespaced class name (see LeftAndMain->Menu()).
Not replacing the old .icon-assetadmin rule since we want the old and new UI to co-exist for a short while.
Ideally we'd move these styles to the asset-admin module, but the whole way of referencing image sprites
is outdated anyway - so I'd prefer to merge this quick fix and think about SVG icons for menu entries instead.
  • Loading branch information
chillu committed Apr 21, 2016
1 parent 9331b40 commit 8c63ae3d68e65e75a8c63efa086858be0eaf9089

Large diffs are not rendered by default.

@@ -19,7 +19,8 @@
&.icon-campaignadmin {
background-position: 0 round(nth(sprite-position($sprite, "collection"), 2) / 2);
}
&.icon-assetadmin {
&.icon-assetadmin,
&.icon-silverstripe-assetadmin-controller-assetadmin {
background-position: 0 round(nth(sprite-position($sprite, "picture"), 2) / 2);
}
&.icon-cmsmain {
@@ -71,7 +71,8 @@
&.icon-campaignadmin {
@include sprite($menu-icons-24x24-collection, inline-block);
}
&.icon-assetadmin {
&.icon-assetadmin,
&.icon-silverstripe-assetadmin-controller-assetadmin {
@include sprite($menu-icons-24x24-picture, inline-block);
}
&.icon-cmsmain {
@@ -105,7 +106,8 @@
&.icon-campaignadmin {
@include sprite($menu-icons-16x16-collection, inline-block);
}
&.icon-assetadmin {
&.icon-assetadmin,
&.icon-silverstripe-assetadmin-controller-assetadmin {
@include sprite($menu-icons-16x16-picture, inline-block);
}
&.icon-cmsmain {

0 comments on commit 8c63ae3

Please sign in to comment.
You can’t perform that action at this time.