lucide
lucide copied to clipboard
feat(icons): added `rose` icon
What is the purpose of this pull request?
- [x] New Icon
- [ ] Bug fix
- [ ] New Feature
- [ ] Documentation update
- [ ] Other:
Description
This comes from the lab
branch.
Icon use case
Anything and everything romantic, i.e. Valentine's Day.
Alternative icon designs
Icon Design Checklist
Concept
- [x] I have provided valid use cases for each icon.
- [x] I have not added any a brand or logo icon.
- [x] I have not used any hate symbols.
- [x] I have not included any religious or political imagery.
Author, credits & license
- [ ] The icons are solely my own creation.
- [x] The icons were originally created in #1398 by @danielbayley
- [ ] I've based them on the following Lucide icons:
- [ ] I've based them on the following design:
Naming
- [x] I've read and followed the naming conventions
- [x] I've named icons by what they are rather than their use case.
- [x] I've provided meta JSON files in
icons/[iconName].json
.
Design
- [x] I've read and followed the icon design guidelines
- [x] I've made sure that the icons look sharp on low DPI displays.
- [x] I've made sure that the icons look consistent with the icon set in size, optical volume and density.
- [x] I've made sure that the icons are visually centered.
- [x] I've correctly optimized all icons to two points of precision.
Before Submitting
- [x] I've read the Contribution Guidelines.
- [x] I've checked if there was an existing PR that solves the same issue.
Added or changed icons
Preview cohesion
Preview stroke widths
DPI Preview (24px)
Icon X-rays
Icons as code
Works for: lucide-react
, lucide-react-native
, lucide-preact
, lucide-vue-next
const RoseIcon = createLucideIcon('Rose', [
["path",{"d":"M13 20s-5 3-9.2-2c0 0 5.2-3 9.2 2"}],
["path",{"d":"m14 2.6-2 1.2a6 6 0 1 0 6 10.4l2-1.2a3 3 0 1 0-4-6.65"}],
["path",{"d":"M18.04 9.637A4.04 4.04 0 1 1 20.025 6.5"}],
["path",{"d":"M9.77 12C4 15 2 22 2 22"}],
["circle",{"cx":"17","cy":"8","r":"2"}]
])
The smallest circle would need to be on the grid for it to be crisp, also, the bottom right path feels a bit wonky:
This corner feels worse
Yeah, I guess, you're right. I can fix it, but we'll need cubic curves:
Open lucide studio
I feel like the reduced roundness of @danielbayley version better.
I feel like the leaf is a bit off-kilter.
Maybe we can steal something from https://lucide.dev/icons/sprout
I feel like the leaf is a bit off-kilter.
Maybe we can steal something from https://lucide.dev/icons/sprout
Maybe, the generic shape is OK, but pixel perfection would need to be improved.
I feel like the leaf is a bit off-kilter. Maybe we can steal something from https://lucide.dev/icons/sprout
Maybe, the generic shape is OK, but pixel perfection would need to be improved.
Do you mean for better sharpness?
I actually feel like your admittedly sharper version feels harsh and reduces the soft feel which I like.
Do you mean for better sharpness?
I actually feel like your admittedly sharper version feels harsh and reduces the soft feel which I like.
Yeah, tastes aside, we have "I've made sure that the icons look sharp on low DPI displays" in our PR checklist for a reason, I actually recall it was in fact none other but yourself who wrote this one. 😄
Do you mean for better sharpness? I actually feel like your admittedly sharper version feels harsh and reduces the soft feel which I like.
Yeah, tastes aside, we have "I've made sure that the icons look sharp on low DPI displays" in our PR checklist for a reason, I actually recall it was in fact none other but yourself who wrote this one. 😄
Fair enough.
If you center the center and keep the outer parts, they are not centered on the grid tho.
If you center the center and keep the outer parts, they are not centered on the grid tho.
I'm not sure what point you're trying to make here, these paths are from your original design, aren't they? Yeah, that's the whole reason I did the redesign.
Icon use case
Anything and everything romantic, i.e. Valentine's Day.
Yeah I was playing around with an animated version for something related, at the time.
There is also rose-2
, to match flower-2
, which maybe should be included here?
Maybe we can steal something from https://lucide.dev/icons/sprout
@jguddas Sure, it was of course already based on leaves from existing floral icons sprout
and flower-2
…
the reduced roundness of @danielbayley version
Yeah, I think bloating it like that distorts the elegant appearance of a rose.
If you center the center and keep the outer parts, they are not centered on the grid tho.
@jguddas @karsa-mistmere I would have ran it though the optimisation step, which although a very useful tool, was sometimes nudging things slightly off-grid… But for reference, here are the grid-aligned circles it was based on, when I originally designed it in Illustrator: Lucide Studio link…
Unfortunately the smallest circle is off grid, hence @jguddas's little red marker:
Unfortunately the smallest circle is off grid, hence @jguddas's little red marker
Nice feature, but yes exactly .5 off… Is that not allowed?
Unfortunately the smallest circle is off grid, hence @jguddas's little red marker
Nice feature, but yes exactly .5 off… Is that not allowed?
If you have are .5 off you will fill the pixel to only 50% which will look not so sharp.
We could make the center circle smaller, than it's more than 50% there.
@jguddas @karsa-mistmere We could do like this…
@jguddas @karsa-mistmere We could do like this…
Having the circle at 1.75
radius still means we have a pixel that is 25%
filled.
I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.
I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.
I disagree, the 8px circles are also super blurry at 1px and would need to alight to the grid more:
I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.
I disagree, the 8px circles are also super blurry at 1px and would need to alight to the grid more:
And again thin or not (although I'd very much prefer a rounder real rose to a plastic one), this wobbly path would need to be fixed:
I think this is the best of all the versions so far, which is just the current suggestion with the center of the circle snapped.
I disagree, the 8px circles are also super blurry at 1px and would need to alight to the grid more
@karsa-mistmere @jguddas How about this?
As an aside, it would be super useful if we could upgrade ‘Studio’ to also (async
hronously) generate the relevant previews at various stroke-width
s, and 100% scale pixel preview… Along with the ability to add temporary ‘guide’ layers (maybe with a .guide
class or something) that aren’t forced to render at 2px stroke-width
, and with a contrasting colour…
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days.
I think the current state of the pr looks sharp enough to merge.