-
-
Notifications
You must be signed in to change notification settings - Fork 554
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
Conversation
@danielbayley is attempting to deploy a commit to the Lucide Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice LGTM!
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 Anyway, the message |
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. |
@jguddas For comparison…
Well bigger than they were… I just think the pointer gets a bit too drowned out by the full 20x20 circle…
Yeah, as mentioned #1208 (comment), I think that’s for a separate PR though… |
@karsa-mistmere you are the expert here, I prefer the bigger one, what do you think? |
@danielbayley Can you rebase the branch? |
Yeah, those are awesome, could be the default so |
3 variants seems like overkill though… I would vote these as default What do you guys think of that? |
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). |
I also prefer the existing ones. So I agree with @karsa-mistmere on this. |
@karsa-mistmere There is actually more space, horizontally… For example, I can’t fit the nice What do we think of @jguddas suggestion to add them simply as Seems a shame to waste them… |
@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 |
There was a problem hiding this 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.
* 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>
The existing
message-circle
is a bit small, for no obvious reason…message-square-warning
andmessage-square-diff
arereport
andrequest-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:
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 existingmessage-circle-*
variants)…