Skip to content
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.

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

Refine/add message[-*] icons #1208

Merged
merged 37 commits into from
Dec 13, 2023
Merged

Conversation

danielbayley
Copy link
Member

@danielbayley danielbayley commented May 3, 2023

The existing message-circle is a bit small, for no obvious reason…

message-square-warning and message-square-diff are report and request-changes from codicons, respectively. The former is also part of octicons.

Some other options… not sure what is desirable to include, so leaving them here:

Untitled

Update: Added message-square-share as part of the set (cross-reference from octicons, as seen above right here on GitHub…)


Edit: message-bubble-* variant (potential replacement for existing message-circle-* variants)…

preview

@vercel
Copy link

vercel bot commented May 3, 2023

@danielbayley is attempting to deploy a commit to the Lucide Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

github-actions bot commented May 3, 2023

Added or changed icons

icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg

Preview cohesion icons/square-slash.svg icons/chevron-down-square.svg icons/message-square-x.svg icons/scissors-square-dashed-bottom.svg icons/arrow-up-left-square.svg icons/mouse-pointer-square-dashed.svg icons/messages-square.svg icons/play-square.svg icons/square-code.svg icons/chevron-right-square.svg icons/minus-square.svg icons/square-equal.svg icons/pi-square.svg icons/arrow-down-right-square.svg icons/square-stack.svg icons/pilcrow-square.svg icons/message-square-heart.svg icons/chevron-up-square.svg icons/chevron-left-square.svg icons/key-square.svg icons/pen-square.svg icons/message-square-diff.svg icons/arrow-left-square.svg icons/message-square-quote.svg
icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg
icons/smartphone-nfc.svg icons/sparkle.svg icons/shield.svg icons/align-vertical-justify-center.svg icons/rat.svg icons/webcam.svg icons/axe.svg icons/chevron-right-circle.svg icons/clock.svg icons/flip-horizontal-2.svg icons/footprints.svg icons/chevrons-right.svg icons/book-copy.svg icons/package-minus.svg icons/bookmark-minus.svg icons/mail-x.svg icons/step-back.svg icons/align-horizontal-distribute-center.svg icons/sliders.svg icons/siren.svg icons/candy.svg icons/bean.svg icons/align-center.svg icons/align-end-horizontal.svg
Preview stroke widths icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg
icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg
icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg
DPI Preview (24px) icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg
Icon X-rays icons/message-circle-code.svg icons/message-circle-dashed.svg icons/message-circle-heart.svg icons/message-circle-more.svg icons/message-circle-off.svg icons/message-circle-plus.svg icons/message-circle-question.svg icons/message-circle-reply.svg icons/message-circle-warning.svg icons/message-circle-x.svg icons/message-circle.svg icons/message-square-code.svg icons/message-square-diff.svg icons/message-square-dot.svg icons/message-square-heart.svg icons/message-square-more.svg icons/message-square-off.svg icons/message-square-plus.svg icons/message-square-quote.svg icons/message-square-reply.svg icons/message-square-share.svg icons/message-square-text.svg icons/message-square-warning.svg icons/message-square-x.svg

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.

Nice LGTM!

icons/message-circle-question.svg Show resolved Hide resolved
icons/message-square-diff.svg Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

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

This one looks out of proportion, if we can't fit a taller exclamation mark somehow (e.g. file-warning) let's just drop it.

Copy link
Member Author

@danielbayley danielbayley May 6, 2023

Choose a reason for hiding this comment

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

That seems drastic. I think it looks recognisable enough as what it is… But possible alternatives anyway:

Untitled

Untitled

@jguddas
Copy link
Member

jguddas commented May 6, 2023

How does it look compared to other circular icons?

@jguddas
Copy link
Member

jguddas commented May 6, 2023

What about giving the first row a try, than we have more cohesion with the other circular icons?
image

@danielbayley
Copy link
Member Author

How does it look compared to other circular icons?

@jguddas Isn’t that what the new Preview cohesion is for…? :) Nice improvement by the way! Is it selecting icons at random to compare? (I haven’t looked over the code…) If not, I think circle[-something] should be in there, obviously based on this…

Anyway, the message circle is larger than it was before, but 1px less radius than circle to accommodate the pointer, bottom left…

@jguddas
Copy link
Member

jguddas commented May 6, 2023

How does it look compared to other circular icons?

@jguddas Isn’t that what the new Preview cohesion is for…? :) Nice improvement by the way! Is it selecting icons at random to compare? (I haven’t looked over the code…) If not, I think circle[-something] should be in there, obviously based on this…

Anyway, the message circle is larger than it was before, but 1px less radius than circle to accommodate the pointer, bottom left…

It still feels a bit small to me.

The cohesion preview has square icons in the first row and random icons in the last row.

And we should really take a look at the question mark for all icons where it is used.

@danielbayley
Copy link
Member Author

danielbayley commented May 6, 2023

What about giving the first row a try

@jguddas For comparison…

Untitled

Untitled

It still feels a bit small to me.

Well bigger than they were… I just think the pointer gets a bit too drowned out by the full 20x20 circle…

And we should really take a look at the question mark for all icons where it is used.

Yeah, as mentioned #1208 (comment), I think that’s for a separate PR though…

@jguddas
Copy link
Member

jguddas commented May 6, 2023

@karsa-mistmere you are the expert here, I prefer the bigger one, what do you think?
image

@ericfennis
Copy link
Member

@danielbayley Can you rebase the branch?

@jguddas
Copy link
Member

jguddas commented Aug 12, 2023

I also have these, which I think are nicer than the current circle-message-* variants…

Yeah, those are awesome, could be the default so message, and the circle variant is message-circle.

@danielbayley
Copy link
Member Author

Yeah, those are awesome, could be the default so message, and the circle variant is message-circle.

3 variants seems like overkill though… I would vote these as default message and massage-square, dropping the current message-circles…

What do you guys think of that?

@karsa-mistmere
Copy link
Member

What do you guys think? @ericfennis @karsa-mistmere @jguddas

To be honest, I very much prefer the existing ones - I think those two (message-square and message-circle) are pretty well-established shapes, and while this one does look nice it is much less universal and I think the reduced dimensions will eventually prove to be just a bit too tight (e.g. the back/forward variants are already pretty dense as compared to what would be ideal).

@ericfennis
Copy link
Member

I also prefer the existing ones. So I agree with @karsa-mistmere on this.

@danielbayley
Copy link
Member Author

reduced dimensions will eventually prove to be just a bit too tight

@karsa-mistmere There is actually more space, horizontally… For example, I can’t fit the nice voicemail/note variant into message-square, or even the current iteration of message-circle (see also #1208 (comment).

What do we think of @jguddas suggestion to add them simply as message?

Seems a shame to waste them…

@karsa-mistmere
Copy link
Member

Sidenote: I did think of maybe "fixing" message-square's visual weight and centre of gravity instead, but I'm not convinced of this either, the previous shape just feels better.
image
image

@danielbayley danielbayley mentioned this pull request Aug 17, 2023
@danielbayley
Copy link
Member Author

Sidenote: I did think of maybe "fixing" message-square's visual weight and centre of gravity instead, but I'm not convinced of this either, the previous shape just feels better.

@karsa-mistmere Maybe slightly—they are nice—but also feels like it technically ought to be the same width as https://lucide.dev/icons/rectangle-horizontal. These don’t look bad… and the extra space does afford message-square-voicemail, which would be a nice addition. See what you guys think… cc: @jguddas @ericfennis

preview

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.

Nice I think this one is almost good to go.
I've doubts about use-cases for message-dot.
And message-square-text-quote is a bit hard to see on 100%. Probably message-square-quote is better suited for this usecase.

Can you remove message-square-text-quote? And provide a use-case for message-dot? I do not approve active states to use dots.

@ericfennis ericfennis merged commit 14c7ea8 into lucide-icons:main Dec 13, 2023
5 of 6 checks passed
@danielbayley danielbayley deleted the messages branch December 14, 2023 18:19
realguse pushed a commit to realguse/lucide that referenced this pull request Dec 25, 2024
* Add `message-square-warning` icon

* Optimise `message-square-plus` icon

* Add `message-square-diff` (request changes) icon

* Refine `message-circle` icon

* Add `message-circle-question` icon

* Add `message-circle-warning` icon

* Add `message-square-x` icon

* Add `message-square-off` icon

* Add `message-circle-plus` icon

* Add `message-circle-x` icon

* Add `message-circle-off` icon

* Refine `message-circle` icons

* Fix `message-circle-question` mark

* Fix `message-square-diff` icon

* Add `message-square-share` icon

* Add `message-square-text` icon

* Refine `message-square-x` icon

* Add `message-square-code` icon

* Improve `message-*` icons metadata

* Add `message-square-more` icon

* Improve `more-horizontal` metadata

* Add `message-circle-more` icon

* Add `message-circle-dashed` icon

* Add `message-square-text-quote` icon

* Add `message-square-quote` icon

* Add `message-square-reply` icon

* Add `message-circle-code` icon

* Add `message-circle-reply` icon

* Add `message-circle-heart` icon

* Add `message-square-heart` icon

* Add `message-square-dot` (unread) icon

* Add `message-square-dot-big` (unread/notification) variant

* Remove message-square-text-quote and rename message-square-dot-big

* Added as contributor in meta data

---------

Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
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