Skip to content

Conversation

@ericfennis
Copy link
Member

Closes #1479

Icon use case

For marine weather applications: For example, to show/indicate tide levels of the sea/waters.

Alternative icon designs

See #1479

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 Tide (Sea level) #1479 by @danielbayley
  • 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 Jul 28, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 28, 2025

Added or changed icons

waves-arrow-downwaves-arrow-up

Preview cohesion square-starsquares-exclude
waves-arrow-downwaves-arrow-up
person-standingreply
Preview stroke widths waves-arrow-downwaves-arrow-up
waves-arrow-downwaves-arrow-up
waves-arrow-downwaves-arrow-up
DPI Preview

16px (shadcn/ui)

waves-arrow-down waves-arrow-up

24px (default)

waves-arrow-down waves-arrow-up

32px (shadcn/ui + retina)

waves-arrow-down waves-arrow-up

48px (default + retina)

waves-arrow-down waves-arrow-up
Icon X-rays waves-arrow-down waves-arrow-up
Icon Diffs waves-arrow-down waves-arrow-up
Icons as code

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

const WavesArrowDownIcon = createLucideIcon('WavesArrowDown', [
  ["path",{"d":"M12 10L12 2"}],
  ["path",{"d":"M16 6L12 10L8 6"}],
  ["path",{"d":"M2 15C2.6 15.5 3.2 16 4.5 16C7 16 7 14 9.5 14C12.1 14 11.9 16 14.5 16C17 16 17 14 19.5 14C20.8 14 21.4 14.5 22 15"}],
  ["path",{"d":"M2 21C2.6 21.5 3.2 22 4.5 22C7 22 7 20 9.5 20C12.1 20 11.9 22 14.5 22C17 22 17 20 19.5 20C20.8 20 21.4 20.5 22 21"}]
])

const WavesArrowUpIcon = createLucideIcon('WavesArrowUp', [
  ["path",{"d":"M12 2v8"}],
  ["path",{"d":"M2 15c.6.5 1.2 1 2.5 1 2.5 0 2.5-2 5-2 2.6 0 2.4 2 5 2 2.5 0 2.5-2 5-2 1.3 0 1.9.5 2.5 1"}],
  ["path",{"d":"M2 21c.6.5 1.2 1 2.5 1 2.5 0 2.5-2 5-2 2.6 0 2.4 2 5 2 2.5 0 2.5-2 5-2 1.3 0 1.9.5 2.5 1"}],
  ["path",{"d":"m8 6 4-4 4 4"}]
])

@karsa-mistmere
Copy link
Member

@ericfennis: The optical volume feels a tad too high, but I'm not sure how that could be resolved. 🫤

@ericfennis
Copy link
Member Author

@karsa-mistmere Too high in comparison with the other icons?

@karsa-mistmere
Copy link
Member

@karsa-mistmere Too high in comparison with the other icons?

Yeah. Same issue as with sunrise and sunset, really. I'm also not a big fan of those. 😅

@github-actions github-actions bot removed the Stale label Sep 12, 2025
github-actions[bot]

This comment was marked as off-topic.

@ericfennis
Copy link
Member Author

@karsa-mistmere I don't see another way, without disturbing consistency and redesigning a lot of icons.
You okay with merging this?

@ericfennis ericfennis merged commit 0f6dc52 into main Nov 23, 2025
7 checks passed
@ericfennis ericfennis deleted the add-wave-arrow-up-down branch November 23, 2025 16:45
ch4og pushed a commit to csmplay/mapban that referenced this pull request Dec 4, 2025
This PR contains the following updates:

| Package | Change | Age | Confidence |
|---|---|---|---|
| [lucide-react](https://lucide.dev) ([source](https://github.com/lucide-icons/lucide/tree/HEAD/packages/lucide-react)) | [`^0.554.0` -> `^0.555.0`](https://renovatebot.com/diffs/npm/lucide-react/0.554.0/0.555.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/lucide-react/0.555.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/lucide-react/0.554.0/0.555.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>lucide-icons/lucide (lucide-react)</summary>

### [`v0.555.0`](https://github.com/lucide-icons/lucide/releases/tag/0.555.0): Version 0.555.0

[Compare Source](lucide-icons/lucide@0.554.0...0.555.0)

#### What's Changed

- fix(icons): changed `calendars` icon by [@&#8203;jguddas](https://github.com/jguddas) in [#&#8203;3795](lucide-icons/lucide#3795)
- fix(docs): correct package name and description for Flutter and Lustre package ([#&#8203;3701](lucide-icons/lucide#3701)) by [@&#8203;epifaniofrancisco](https://github.com/epifaniofrancisco) in [#&#8203;3703](lucide-icons/lucide#3703)
- feat(angular): Angular V21 Support by [@&#8203;JeevanMahesha](https://github.com/JeevanMahesha) in [#&#8203;3807](lucide-icons/lucide#3807)
- chore(metadata): Adjust navigation category by [@&#8203;ericfennis](https://github.com/ericfennis) in [#&#8203;3461](lucide-icons/lucide#3461)
- feat(icons): Add `waves-arrow-up` and `waves-arrow-down` by [@&#8203;ericfennis](https://github.com/ericfennis) in [#&#8203;3463](lucide-icons/lucide#3463)
- fix(icons): changed `scale` icon by [@&#8203;jamiemlaw](https://github.com/jamiemlaw) in [#&#8203;3800](lucide-icons/lucide#3800)
- feat(icons): added `form` icon by [@&#8203;jguddas](https://github.com/jguddas) in [#&#8203;3558](lucide-icons/lucide#3558)

**Full Changelog**: <lucide-icons/lucide@0.554.0...0.555.0>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0Mi4xNC4yIiwidXBkYXRlZEluVmVyIjoiNDIuMTQuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Reviewed-on: https://git.csmpro.ru/csmpro/mapban/pulls/58
Co-authored-by: Renovate Bot <[email protected]>
Co-committed-by: Renovate Bot <[email protected]>
karsa-mistmere pushed a commit that referenced this pull request Dec 12, 2025
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.

Tide (Sea level)

4 participants