-
-
Notifications
You must be signed in to change notification settings - Fork 511
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
Add ambulance
icon
#1819
Add ambulance
icon
#1819
Conversation
Co-authored-by: Jakob Guddas <github@jguddas.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jordan808 Nice icon, I like it!
For consistency, maybe an idea to update the truck
design as well.
Maybe this is also a better shape to use in #1803
I feel like it's okay for the ambulance to be different and more boxy, that's also how they are in reality. |
@jguddas But I think the shape of the "truck" of @jordan808 fits way better with the lucide style, it's also vertically centered. |
Good point. |
Below, I've explored some options to achieve greater consistency. The last one is my favorite, where the truck utilizes the same cab style as the ambulance icon in this PR. This choice would also be more modern, as the flat-face cab style (known as cab-overs or flat faces) has become less common in recent decades due to truck size regulations, at least in the U.S. Here are some of my explorations aimed at enhancing consistency: What do you all think @jguddas @ericfennis ? |
@jordan808 Nice work on the designs. It's hard to see differences. But I think I like the first and the third options. It uses the space on the canvas a bit better. |
Do you all want to go with the first option then? If so, we would need to increase the cab height of the Proposed change to
|
@jordan808 looks excellent 🤩. I'm in for the new truck icon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added contributors 👍
@jordan808 Can you make a PR for the changes for the |
What is the purpose of this pull request?
Description
Adds ambulance icon as requested by #119. Closes feather icons issue feathericons/feather#1177.
Icon use case
For navigation applications directing users to the nearest emergency services, or for listing an emergency contact.
Alternative icon designs
Icon Design Checklist
Concept
Author, credits & license
Naming
icons/[iconName].json
.Design
Before Submitting