Skip to content

Conversation

@jamiemlaw
Copy link
Contributor

@jamiemlaw jamiemlaw commented Mar 28, 2025

Goes part-way to closing #119

What is the purpose of this pull request?

  • New Icon
  • Bug fix
  • New Feature
  • Documentation update
  • Other:

Description

Adds an hd icon

Icon use case

  • In a media scrubber bar, shows that the item currently playing is in HD
  • On a list of videos, shows that a video is available in HD

Alternative icon designs

icons
Open lucide studio

icons
Open lucide studio

icons
Open lucide studio

My main submission follows the same visual format at the currently-proposed closed captions icon. However, if we were in future to add an SD counterpart to this icon, there would not be enough space for the 'S'. For that reason, it might be worth choosing one of the last two alternatives as a way to future-proof things. If that ends up being the case, would the closed caption icon need to be revised, to have the same size of bounding rectangle?

Icon Design Checklist

Concept

  • I have provided valid use cases for each icon.
  • I have not added any a brand or logo icon.
  • I have not used any hate symbols.
  • I have not included any religious or political imagery.

Author, credits & license

  • The icons are solely my own creation.
  • The icons were originally created in feathericons/#557 by @ahtohbi4
  • I've based them on the following Lucide icons:
  • I've based them on the following design:

Naming

  • I've read and followed the naming conventions
  • I've named icons by what they are rather than their use case.
  • I've provided meta JSON files in icons/[iconName].json.

Design

  • I've read and followed the icon design guidelines
  • I've made sure that the icons look sharp on low DPI displays.
  • I've made sure that the icons look consistent with the icon set in size, optical volume and density.
  • I've made sure that the icons are visually centered.
  • I've correctly optimized all icons to three points of precision.

Before Submitting

@github-actions github-actions bot added 🎨 icon About new icons 🫧 metadata Improved metadata labels Mar 28, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Mar 28, 2025

Added or changed icons

hd

Preview cohesion square-code
hd
grid-3x2
Preview stroke widths hd
hd
hd
DPI Preview

16px (shadcn/ui)

hd

24px (default)

hd

32px (shadcn/ui + retina)

hd

48px (default + retina)

hd
Icon X-rays hd
Icon Diffs hd
Icon Symmetry

Flip Horizontal

hd

Flip Vertical

hd

Flip Diagonal

hd

Rotate 180°

hd
Icons as code

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

const HdIcon = createLucideIcon('Hd', [
  ["path",{"d":"M10 12H6"}],
  ["path",{"d":"M10 15V9"}],
  ["path",{"d":"M14 14.5a.5.5 0 0 0 .5.5h1a2.5 2.5 0 0 0 2.5-2.5v-1A2.5 2.5 0 0 0 15.5 9h-1a.5.5 0 0 0-.5.5z"}],
  ["path",{"d":"M6 15V9"}],
  ["rect",{"x":"2","y":"5","width":"20","height":"14","rx":"2"}]
])

@jamiemlaw jamiemlaw changed the title Add hd icon feat(icons): added hd icon Mar 28, 2025
@jguddas
Copy link
Member

jguddas commented Mar 28, 2025

icons
Open lucide studio

@jamiemlaw
Copy link
Contributor Author

iconsOpen lucide studio

At that size, the 1px rounding just makes the D look more like an O, in my opinion. And having no rounding would keep consistency with https://lucide.dev/icons/case-upper.

@karsa-mistmere
Copy link
Member

This is pretty much identical to one of my hd icon drafts, the reason I never got around to submitting it is that I've been also trying to make related icons work (closed captions, audio description, small definition, 4K, 8K, 5K et cetera):
image

The issue is, while I prefer the top row (which this one is "part of"), that's the only one that's not actually Lucide-compliant. 😅

@jamiemlaw
Copy link
Contributor Author

@karsa-mistmere Yup, I think you and I have been toying with the same ideas!

@jguddas posted a design for closed captions a couple of weeks ago that I really liked, so I tried applying it to the other icons in the set.

image

@karsa-mistmere
Copy link
Member

I don't think HD is all that relevant anymore, tbh. If we cannot make 4K, 5K, 8K etc work, I vote to just skip this.

@karsa-mistmere
Copy link
Member

karsa-mistmere commented Aug 4, 2025

I don't think HD is all that relevant anymore, tbh. If we cannot make 4K, 5K, 8K etc work, I vote to just skip this.

Alternatively, if #3483 is voted to go through, we could add similarly themed icons for these, that are very feasible:
image

Edit: same goes for PDF

@ericfennis
Copy link
Member

@karsa-mistmere I do not agree, it is still used in other applications like images and audio. Where the term 4K doesn't exist.

github-actions[bot]

This comment was marked as resolved.

@jguddas
Copy link
Member

jguddas commented Oct 16, 2025

Maybe we should arcify the sharp corners with a .5 radius?

@ericfennis
Copy link
Member

@jguddas Hmm disagree, the current radius matches the monitor shape.
image

@jguddas
Copy link
Member

jguddas commented Oct 24, 2025

@jguddas Hmm disagree, the current radius matches the monitor shape.
image

I mean the D not having any round corners.

@ericfennis
Copy link
Member

ericfennis commented Oct 24, 2025

@jguddas
Ah, yeah, if we are sure the letter D is still readable and not confused with another letter. But to me it looks still readable, what do you think?

Without
image

With (0.5 radius)
image

@jamiemlaw
Copy link
Contributor Author

I'm happy with adding a .5px radius.

I am on travels so expect a couple of days before I amend the PR.

@ericfennis
Copy link
Member

@jamiemlaw Agreed!

@karsa-mistmere
Copy link
Member

@jguddas Ah, yeah, if we are sure the letter D is still readable and not confused with another letter. But to me it looks still readable, what do you think?

Without image

With (0.5 radius) image

Not bad, but maybe we could enunciate the arcs on the right side a just tiny bit more?

image

icons
Open lucide studio

@ericfennis ericfennis merged commit 0f732b4 into lucide-icons:main Dec 10, 2025
7 checks passed
karsa-mistmere pushed a commit that referenced this pull request Dec 12, 2025
* Add `hd` icon

* Update hd.svg

* Update hd.json

---------

Co-authored-by: Eric Fennis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 icon About new icons 🫧 metadata Improved metadata

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants