Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove artefact in debug icon #2

Closed
wants to merge 3 commits into from
Closed

Remove artefact in debug icon #2

wants to merge 3 commits into from

Conversation

shiftgeist
Copy link

@shiftgeist shiftgeist commented Aug 20, 2019

Artefact that got removed can be seen here (figma).

And rename Debug icon to lowercase debug.

@msftclas
Copy link

msftclas commented Aug 20, 2019

CLA assistant check
All CLA requirements met.

@waldyrious
Copy link

Since you added the case change in a separate commit, I'd suggest also renaming the Search.svg to search.svg.

@miguelsolorio
Copy link
Contributor

@shiftgeist thanks for this, I fixed the missing reference in the Figma file and I also fixed the naming issue (weird git issue that didn't show up on my local machine). Next time I would export from the Figma file into this repo so that the changes are the same (your changes removed the fill colors). Thanks for reporting this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants