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

Patch/fix sidebar template icons #280

Merged
merged 5 commits into from
Jul 20, 2014

Conversation

nanoant
Copy link

@nanoant nanoant commented Jan 28, 2014

  1. Removes unnecessary highlighted sidebar image versions and make all other to use plain black (alpha only)
  2. Removed unused icon source files (this are anyway accessible in Git history)
  3. Uses NSImageCell for applying actual effect, which makes proper proper overlay effect depending on platform
  4. Introduces pixel aligned PDF (scale independent icons) icons that preserve style of @afh previous commit, however give SVG source (instead Pixelmator which is proprietary), for details see cd94264
  5. Uses SVG pixel aligned icons for history file icons, which makes them look sharp on all displays

@nanoant
Copy link
Author

nanoant commented Jan 31, 2014

Here I am attaching preview of how it looked before and how it looks when applying changes above.

Make sure you click on Before & Now and switch tabs to compare.

Before

old alexis

Now

new pdf

Now (Retina)

new pdf retina

@rowanj
Copy link
Owner

rowanj commented May 16, 2014

Looks great; I'm excited to merge this when I've got a few more of the 0.15 launch bugs sorted out.

👍

This removes need for own rendered highlighted sidebar icon.
This is replacement for icons by Alexis Hildebrandt. I tried to be close as
possible to Alexis design, however there are following changes:

(1) Icons are now PDF, no need for @2x version, less files
(2) SVG source, instead Pixelmator which is proprietary paid software
(3) Icons are pixel aligned to non-Retina pixel boundaries (look sharp)
(4) Folder & remote (cloud) look are more inline with OSX default icons
Also removed old Acorn sources and Pixelmator sources. Icons were made to
preserve style of previous ones. No need for source as icons are SVG.
@nanoant
Copy link
Author

nanoant commented Jun 9, 2014

Rebased to latest master.

@rowanj rowanj merged commit 1ecf781 into rowanj:master Jul 20, 2014
rowanj added a commit that referenced this pull request Jul 20, 2014
@rowanj
Copy link
Owner

rowanj commented Jul 20, 2014

Thanks for the rebase; love it!

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.

2 participants