lucide icon indicating copy to clipboard operation
lucide copied to clipboard

fix(icons): update `pentagon` icon

Open jguddas opened this issue 11 months ago • 5 comments

What is the purpose of this pull request?

  • [x] Other: Icon update

Description

Hand optimised corners to have 2px radius.

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 Feb 24 '24 20:02 jguddas

Added or changed icons

icons/pentagon.svg
Preview cohesion icons/square-library.svg
icons/pentagon.svg
icons/square-library.svg
Preview stroke widths icons/pentagon.svg
icons/pentagon.svg
icons/pentagon.svg
DPI Preview (24px) icons/pentagon.svg
Icon X-rays icons/pentagon.svg
Icons as code

Works for: lucide-react, lucide-react-native, lucide-preact, lucide-vue-next

const PentagonIcon = createLucideIcon('Pentagon', [
  ["path",{"d":"M10.83 2.38a2 2 0 0 1 2.34 0l8 5.74a2 2 0 0 1 .73 2.25l-3.04 9.26a2 2 0 0 1-1.9 1.37H7.04a2 2 0 0 1-1.9-1.37L2.1 10.37a2 2 0 0 1 .73-2.25z"}]
])

github-actions[bot] avatar Feb 24 '24 20:02 github-actions[bot]

I'm not super happy about the optical volume and centre of gravity of this icon, I feel like maybe it should be scaled up towards the top as such: image

What do you think?

karsa-mistmere avatar Feb 25 '24 09:02 karsa-mistmere

@karsa-mistmere To me it is not really centered if you place the icon in a single form.

ericfennis avatar Mar 06 '24 19:03 ericfennis

@karsa-mistmere To me it is not really centered if you place the icon in a single form.

I find it to be a bit too bottom-heavy. 🤔

image (top row current, bottom row proposal)

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

@karsa-mistmere ah, seeing this. I think I'm going with your proposal

ericfennis avatar Mar 07 '24 07:03 ericfennis