lucide icon indicating copy to clipboard operation
lucide copied to clipboard

Remove brand icons?

Open moeenio opened this issue 4 years ago • 17 comments

Since we don't add any brand icons, I think we should remove the current ones. What do you think?

moeenio avatar Oct 13 '20 15:10 moeenio

I agree

ghost avatar Oct 14 '20 16:10 ghost

I agree, maybe we can move the current brand icons to a separate repo. I can imagine people want to use it

ericfennis avatar Oct 16 '20 09:10 ericfennis

@locness3 @delnyn What about the idea, to create a separate package in packges folder called: 'lucide-brands'. We can move all the brand icons to this separate package, if people like to have this type of brand icons they can use the package. In the readme we can notice that the brand icons are deprecated and we don't update/maintain them.

ericfennis avatar Dec 20 '20 15:12 ericfennis

Oh yes! Should I clone this one and remove the icons?

ghost avatar Dec 20 '20 21:12 ghost

I don't know, but I think it'd be best if people didn't use them.

moeenio avatar Dec 21 '20 07:12 moeenio

How about create deprecated section for icons instead of completely removing them?

mittalyashu avatar Dec 29 '20 10:12 mittalyashu

Just for the record, this is/was also a concern in the Feather project, where is stated that

simple-icons/simple-icons repository already covers everything, uses SVG and the same 24x24 viewBox, so it could be a great replacement for people that need brand icons.

dgdavid avatar Dec 29 '20 13:12 dgdavid

Agreed.

mikeott avatar May 21 '21 00:05 mikeott

soooo? we just delete them all?

ghost avatar Mar 28 '22 21:03 ghost

I'm thinking of defining the ultimate v1.0 for Lucide and I think this should be a part of it.

ericfennis avatar Mar 29 '22 19:03 ericfennis

what do you mean by "defining the ultimate v1.0 for Lucide"?

ghost avatar Mar 30 '22 18:03 ghost

We are going with a lot of requests to add icons. I'm think split project. Part for medical, part for programing, part for brand , ... . In this case, 'quality, version, update, ...' of each package are different. In angular, we can one project for handle of use multi package icon library.

SMAH1 avatar Apr 09 '22 03:04 SMAH1

I don't understand lol

ghost avatar Apr 11 '22 06:04 ghost

what do you mean by "defining the ultimate v1.0 for Lucide"? @delnyn I will create a separate discussion for that later. I need write down some points for that.

ericfennis avatar Apr 11 '22 12:04 ericfennis

@SMAH1 Hmm I'm not fan of this idea. I think it wil add some unnecessary complexity. And can be really confusion as well.

ericfennis avatar Apr 11 '22 12:04 ericfennis

~~Yeah, I agree, and apart from adding a layer of unnecessary complexity, this would introduce the problem of having to deal with icons overlapping multiple categories. Categorising icons sounds like a potentially useful feature, but it's partly accomplished already via tagging. I would maybe add very basic filtering on tags to the lucide.dev UI and make sure that icon categories are properly filled in within icon tags.~~

Edit: I have changed my mind about categories, see: #669

karsa-mistmere avatar Apr 18 '22 19:04 karsa-mistmere

Hello guys,

If I can offer my 2 cents here, I do think it's fine to just remove them all-together. There are other alternatives so it's fine, there's no need to overcomplicate things just for some existing icons.

PodaruDragos avatar May 31 '22 12:05 PodaruDragos

if I may chime in, I agree. They should be removed, maybe moved to a seperate lucide-brands or depreciated repository--where people would be able to add more brand icons, if they wanted. But they wouldn't be part of the main lucide.

Especially since, due to lucides design guidelines, it's hard to accurately represent the logos while keeping them both recigniseable, good looking, and representative of the logo.

tryoxiss avatar Jan 15 '23 07:01 tryoxiss

yeah I think everyone agrees. this issue is literally older than my brother, please someone remove the brand icons

ghost avatar Jan 23 '23 23:01 ghost

Interesting

moeenio avatar Jan 24 '23 19:01 moeenio

I think this is something we should do for lucide 1.0 and create a separate package, like @tryoxiss mentions.

ericfennis avatar Jan 24 '23 20:01 ericfennis

From what I understand, these brand icons were removed, and the only way to get them now is to use simple-icons?

Sawtaytoes avatar Jun 28 '23 08:06 Sawtaytoes

From what I understand, these brand icons were removed, and the only way to get them now is to use simple-icons?

@Sawtaytoes They haven’t actually been removed yet, but will be. In any case, I will be maintaining my own lucide-brands fork retaining them and some more…

danielbayley avatar Jun 28 '23 10:06 danielbayley

I understand the logic for not adding any new brand icons, but I'd be against removing the current ones from the package. They fit in really well with the design language of the other icons, and make it easier for users to get started with just the single icon library.

tschoffelen avatar Jul 01 '23 16:07 tschoffelen

From what I understand, these brand icons were removed, and the only way to get them now is to use simple-icons?

@Sawtaytoes They haven’t actually been removed yet, but will be. In any case, I will be maintaining my own lucide-brands fork retaining them and some more…

A lucide-brands package would be great. I want to keep using the same style for all the icons, but I need brand icons especially for social media stuff. If a new package emerges I will try to contribute.

devlulcas avatar Oct 13 '23 14:10 devlulcas

@devlulcas They currently exist on the brands branch of my fork here: https://github.com/danielbayley/lucide/commits/brands. I haven’t decided exactly what to do with them in a more canonical way, yet… There are also a growing number of icons from rejected PRs here, which also should live somewhere. Possibly something like lucide-brands, and/or lucide-extra[s] packages… But need an easy way to hook into Lucide to add them to the fonts…

danielbayley avatar Oct 13 '23 17:10 danielbayley