Skip to content

feat(icons): added brick-wall-fire icon #3036

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 10 commits into from
Apr 18, 2025
Merged

feat(icons): added brick-wall-fire icon #3036

merged 10 commits into from
Apr 18, 2025

Conversation

karsa-mistmere
Copy link
Member

closes #2770

What is the purpose of this pull request?

  • New Icon

Description

Added new brick-wall-fire icon.

Icon use case

To represent firewalls, both in a literal and figurative sense, e.g.

  • on a cybersecurity platform to represent firewalls, their rules, or configuration
  • for mobile device network settings
  • on network device configuration panels
  • on network architecture diagrams
  • in a SOC UI to indicate a rule was triggered
  • etc

Alternative icon designs

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 Firewall #2770 by @karsa-mistmere
  • I've based them on the following Lucide icons: brick-wall, flame-kindling
  • 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 Apr 14, 2025
@github-actions GitHub Actions
Copy link
Contributor

Added or changed icons

icons/brick-wall-fire.svg

Preview cohesion icons/key-square.svg
icons/brick-wall-fire.svg
icons/file-json-2.svg
Preview stroke widths icons/brick-wall-fire.svg
icons/brick-wall-fire.svg
icons/brick-wall-fire.svg
DPI Preview (24px) icons/brick-wall-fire.svg
Icon X-rays icons/brick-wall-fire.svg
Icon Diffs icons/brick-wall-fire.svg
Icons as code

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

const BrickWallFireIcon = createLucideIcon('BrickWallFire', [
  ["path",{"d":"M16 3v2.107"}],
  ["path",{"d":"M17 9c1 3 2.5 3.5 3.5 4.5A5 5 0 0 1 22 17a5 5 0 0 1-10 0c0-.3 0-.6.1-.9a2 2 0 1 0 3.3-2C13 11.5 16 9 17 9"}],
  ["path",{"d":"M21 8.274V5a2 2 0 0 0-2-2H5a2 2 0 0 0-2 2v14a2 2 0 0 0 2 2h3.938"}],
  ["path",{"d":"M3 15h5.253"}],
  ["path",{"d":"M3 9h8.228"}],
  ["path",{"d":"M8 15v6"}],
  ["path",{"d":"M8 3v6"}]
])

ericfennis
ericfennis approved these changes Apr 18, 2025
View reviewed changes
Copy link
Member

@ericfennis ericfennis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coolll!

brick-wall-shield Is maybe also cool one to add as well. Aligning the fire-wall usecase

@ericfennis ericfennis merged commit 6eb67bd into lucide-icons:main Apr 18, 2025
9 checks passed
@karsa-mistmere
Copy link
Member Author

Coolll!

brick-wall-shield Is maybe also cool one to add as well. Aligning the fire-wall usecase

@ericfennis: How about this?

icons
Open lucide studio

@ericfennis
Copy link
Member

@karsa-mistmere Exactly what I had in mind!

jpjacobpadilla pushed a commit to briz123/lucide that referenced this pull request Apr 18, 2025
@karsa-mistmere @jpjacobpadilla
feat(icons): added brick-wall-fire icon (lucide-icons#3036)
2c64db1 
* [github] Added issue template forms

* [github] yaml => yml

* Syntax fixes

* Further syntax fixes

* Sort issue templates

* Update 02_bug_report.yml

* Added icons/brick-wall-fire.svg

* Added icons/brick-wall-fire.json

---------

Co-authored-by: Karsa <karsa@karsa.org>
@renovate renovate bot mentioned this pull request May 12, 2025
Closed
1 task
@renovate renovate bot mentioned this pull request Jul 29, 2025
Open
1 task
@karsa-mistmere karsa-mistmere mentioned this pull request Jul 29, 2025
Merged
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericfennis ericfennis ericfennis approved these changes

@jguddas jguddas jguddas approved these changes

Assignees
No one assigned
Labels
🎨 icon About new icons 🫧 metadata Improved metadata
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Firewall
3 participants
@karsa-mistmere @ericfennis @jguddas