Change icon on admin menu

[ghost]

All Submissions:

• Have you followed the ClassicCommerce Contributing guideline?
• Does your code follow the repo standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Change the icon on the main admin menu. This is a temporary fix until we get our own font set. It uses one of the standard CMS dashicons (shopping cart).

How to test the changes in this Pull Request:

1. Copy the revised menu.css file into a test site into assets > css
2. Visit admin area of website

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

Changelog entry

Change the admin main menu icon to remove Woo icon.

[bahiirwa]

What happened to this consesus? https://forums.classicpress.net/t/classiccommerce-logo/647/29 and https://forums.classicpress.net/t/classiccommerce-logo/647/42

[ghost]

This is just a temporary fix to get rid of Woo.

We have a logo and possible font. But we need someone to make up the whole works. All we have at this stage is one or two large images. No icons, etc. Should we approach Michelle and ask her if she has time to do something for us?

[bahiirwa]

It's worth a try. I know we need to make this an SVG or PNG to hook it up in the icons. I think we should nail the branding task in one swoop so we don't leave any bad marks behind.

I wonder if CP will have some more dashicons too in the near future.(wild thoughts)

[ghost]

OK - will ask Michelle for advice.

[ghost]

Michelle happy to help but needs the vector files. I have contacted Viktor (who did the original icon design) to see if he can supply them.

[bahiirwa]

Note: The .scss file compiles the .css file. Please make changes in the .scss file and compile the css using the instructions in the contributions instructions.

You will need to have node installed to run npm install && npm run build in your terminal after making the scss changes. This will match our build process. If you find issues here, I am willing to help push this.

[ghost]

Still waiting to hear from Michelle re logos. This PR was just an interim solution, but I agree that we should get this sorted properly. Also note that I don't use node and have no idea what it does. And very vague on .scss too! Suggest we close this PR.

[bahiirwa]

No, let’s keep it open. We will work through it.

[bahiirwa]

@simplycomputing This is also broken as the branch is missing. Will you submit a new PR?

[ghost]

We can delete this as we will have to do something completely different if we are going with the svg for the dashicon.