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

Update toolbar icon to match VS Code outline style #105

Merged
merged 5 commits into from
Nov 18, 2019
Merged

Conversation

yhatt
Copy link
Member

@yhatt yhatt commented Nov 17, 2019

I'm trying to match style of added toolbar icon to VS Code's outline style (microsoft/vscode-icons).

A previous colored icon helps to make attention user. However, it was too much standing out from the other toolbar buttons. By matching style, we would get a seamless experience familiar with VS Code.

Current New (Outline only) New (with brand color)
Dark theme
Light theme

It looks like Marpit rather than Marp, but I don't think it loses a brand because we are going to keep using colored icon as the extension icon. In fact, SVG icons for marp-vscode has detailed differences from Marp and Marpit, for matching style to VS Code.

Marp brand Marpit logo Marp outline
(marp-cli)
Marp outline
(marp-vscode)

Unlike Marpit and an outline icon for Marp CLI, VS Code's outline icon has bevel corners to match vscode-icons style.

My concern is the outline icon with monochrome stroke may make finding out Marp icon difficult. So I also created another version colored with our primary / secondary brand color. We would like to want feedbacks from extension user.

ToDo

  • Update screenshot in README.md to apply new outline icon

images/icon-dark.svg Outdated Show resolved Hide resolved
@yhatt yhatt merged commit 9c7e6c7 into master Nov 18, 2019
@yhatt yhatt deleted the update-toolbar-icon branch November 18, 2019 15:32
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.

1 participant