lucide icon indicating copy to clipboard operation
lucide copied to clipboard

feat(icons): added `rose` icon

Open jguddas opened this issue 11 months ago • 25 comments

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

image

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.

jguddas avatar Mar 10 '24 16:03 jguddas

Added or changed icons

icons/rose.svg
Preview cohesion icons/square-check-big.svg
icons/rose.svg
icons/guitar.svg
Preview stroke widths icons/rose.svg
icons/rose.svg
icons/rose.svg
DPI Preview (24px) icons/rose.svg
Icon X-rays icons/rose.svg
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"}]
])

github-actions[bot] avatar Mar 10 '24 16:03 github-actions[bot]

The smallest circle would need to be on the grid for it to be crisp, also, the bottom right path feels a bit wonky: image

karsa-mistmere avatar Mar 10 '24 21:03 karsa-mistmere

A bit more geometric redesign that's crisper at 1x: image

icons
Open lucide studio

karsa-mistmere avatar Mar 11 '24 07:03 karsa-mistmere

This corner feels worse Screenshot 2024-03-11 at 13 53 43

jguddas avatar Mar 11 '24 12:03 jguddas

Yeah, I guess, you're right. I can fix it, but we'll need cubic curves: icons
Open lucide studio

karsa-mistmere avatar Mar 11 '24 13:03 karsa-mistmere

I feel like the reduced roundness of @danielbayley version better.

image

jguddas avatar Mar 11 '24 16:03 jguddas

I feel like the leaf is a bit off-kilter.

Maybe we can steal something from https://lucide.dev/icons/sprout

jguddas avatar Mar 11 '24 16:03 jguddas

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.

karsa-mistmere avatar Mar 11 '24 16:03 karsa-mistmere

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.

image

jguddas avatar Mar 11 '24 17:03 jguddas

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. 😄

karsa-mistmere avatar Mar 11 '24 17:03 karsa-mistmere

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.

icons
Open lucide studio

jguddas avatar Mar 11 '24 18:03 jguddas

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.

karsa-mistmere avatar Mar 12 '24 07:03 karsa-mistmere

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

danielbayley avatar Mar 12 '24 15:03 danielbayley

Unfortunately the smallest circle is off grid, hence @jguddas's little red marker: image

karsa-mistmere avatar Mar 12 '24 15:03 karsa-mistmere

Unfortunately the smallest circle is off grid, hence @jguddas's little red marker

Nice feature, but yes exactly .5 off… Is that not allowed?

danielbayley avatar Mar 12 '24 15:03 danielbayley

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.

icons
Open lucide studio

jguddas avatar Mar 12 '24 15:03 jguddas

@jguddas @karsa-mistmere We could do like this

danielbayley avatar Mar 12 '24 15:03 danielbayley

@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.

jguddas avatar Mar 12 '24 16:03 jguddas

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.

icons
Open lucide studio

jguddas avatar Mar 12 '24 16:03 jguddas

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: image

karsa-mistmere avatar Mar 12 '24 16:03 karsa-mistmere

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: image

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: image

karsa-mistmere avatar Mar 12 '24 16:03 karsa-mistmere

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 (asynchronously) generate the relevant previews at various stroke-widths, 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…

danielbayley avatar Mar 12 '24 16:03 danielbayley

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.

github-actions[bot] avatar May 10 '24 02:05 github-actions[bot]

I think the current state of the pr looks sharp enough to merge.

jguddas avatar May 11 '24 16:05 jguddas