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

Icon Font v1.1 #837

Closed
andrew-frueh opened this issue Apr 23, 2019 · 11 comments · Fixed by #840
Closed

Icon Font v1.1 #837

andrew-frueh opened this issue Apr 23, 2019 · 11 comments · Fixed by #840
Assignees

Comments

@andrew-frueh
Copy link
Collaborator

Adding this issue as a place to collect suggestions and comments on the Catalyst icon set.

https://cashmere.healthcatalyst.net/styles/icons

My intention is to update the font in versions as we would with any component, so it will be clear to all team members what and when is changing in our iconography.

@andrew-frueh andrew-frueh self-assigned this Apr 23, 2019
@andrew-frueh
Copy link
Collaborator Author

Suggestion for hc-medical

"One very, very minor criticism I have is that it uses the Caduceus (rod with two snakes and wings) for the symbol of medicine. Enough organizations have confused this for the Rod of Asclepius (rod with one snake) that they've become synonymous in many people's minds, but the latter is the correct symbol."

http://www.kristenelisephd.com/2013/10/what-difference-snake-makes-caduceus.html
https://www.premiumcaremd.com/blog/the-battle-of-the-snakes-staff-of-aesculapius-vs-caduceus

@andrew-frueh
Copy link
Collaborator Author

Suggestion for addition

Include the unique Catalyst filters icon. It's currently in a CSS class here:
https://cashmere.healthcatalyst.net/components/drawer/examples

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

Suggestion on folder icons:
I often use the closed and open folders as buttons that toggle back and forth, so its nice to have them set up in a way that switching between them is nice and smooth. I'm worried the new folder icons won't look quite as good when toggling as the icons I'm currently using.

folder-icons

(current icons on the left, new hc-icons on the right)

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

PopBuilder's population doesn't feel like it fits super well with the other db icons:
image
image

There is def a case where some icons that are solid (filled in) work better in certain scenarios (like in popbuilder, we use the population icon inside of a colored circle. So maybe its fine? Just something that jumped out at me.

image

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

Another thought, from a comment I saw about the skip icon. Maybe we could something that looks more like the queued icon.
image

Bottom left was my quick and dirty, could make something that look alot nicer.

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

Could these clipboards be made look a more similar?
image

image

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

Looks like these strokes need to be merged:
image

@corykon
Copy link
Collaborator

corykon commented Apr 23, 2019

hc-patient and hc-doctor are very similar in shape, but not quite the same
image
image

@andrew-frueh
Copy link
Collaborator Author

Include app switcher icon that we're using in our hc-navbar

@andrew-frueh
Copy link
Collaborator Author

andrew-frueh commented Apr 24, 2019

Health Catalyst's commonly used Data Operating System illustration includes several concepts that we're touching on in this icon set, and a few others that should be considered:
Data-Operating-System-DOS-1024x491

andrew-frueh added a commit to andrew-frueh/Fabric.Cashmere that referenced this issue Apr 24, 2019
updates/additions to the hcicon set

closes HealthCatalyst#837
@benjanderson
Copy link
Contributor

🎉 This issue has been resolved in version 5.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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 a pull request may close this issue.

3 participants