etn-ccis / blui-icons

Brightlayer UI icons for Eaton applications. Includes design files, icon font, and svg icons
BSD 3-Clause "New" or "Revised" License
9 stars 7 forks source link

Add 33 new icons #269

Closed huayunh closed 1 year ago

huayunh commented 1 year ago

~I need technical help! I cannot seem to make my new icons show up in the demo projects!~ Fixed now

New icons added

Pliers, Screwdriver, Box Cutter, Hammer, Home Repair Service Outline, TV Remote, Angular, React, GitHub, Database, Bolt Off, PPE Level, PPE Level Two Tone, Multimeter, Multimeter Outline, Blast, Arc Rated Hood, Balaclava, Overalls, Arc Rated Suit, Glove, Glove Alt, Insulated Glove, Insulated Glove Alt, Left Outline, Right Outline, Left, Right, Double Triangle, Screw Head Slotted, Screw Head Phillips, Heat Shrink Tube, Hard Hat

image

joebochill commented 1 year ago

To get the icons to show in the demo, you'll need to make sure that the versions in the package.json of the demo will match the versions in the package.json of the icon packages. It looks like neither have been updated in this PR.

joebochill commented 1 year ago

I would also strongly recommend against putting two tones icons in the package...they will not work at all as an icon font. I think we'll probably want to consider setting up a different package if we want to have a library of two-tones.

JeffGreiner-eaton commented 1 year ago

I verified the new icons and had questions. is bolt_off correct name? or should it be volt_off is the ppe_levels correct name? or should it be ippe_levels? just checking... One more thing, we have 3 two_tones in here, are we leaving them in?

also we have 391 file changes in here for the 32ish icons?

huayunh commented 1 year ago