Skip to content
This repository has been archived by the owner on Jan 31, 2023. It is now read-only.

Ellipsis #180

Closed
ashygee opened this issue Jan 28, 2020 · 12 comments
Closed

Ellipsis #180

ashygee opened this issue Jan 28, 2020 · 12 comments

Comments

@ashygee
Copy link
Contributor

ashygee commented Jan 28, 2020

No description provided.

@edokoa
Copy link
Contributor

edokoa commented Jan 29, 2020

We have the normal vs. inverted versions. From the explorations I made, those are the sizes and distances that produce clean results. I think the left one doesn't work so well because the dots look too close, the inverted one has the same distance, but being white on black fixes a little bit the distance problem.

https://www.figma.com/file/2rjsQp42uKvzPXzLZIqOeA/Octicons-v2?node-id=255%3A512

image

@edokoa
Copy link
Contributor

edokoa commented Jan 29, 2020

Current version:
image

V1
image

V2
image

V3
image

@edokoa
Copy link
Contributor

edokoa commented Jan 29, 2020

This icon is originally 12px. Should we make it in 24 instead of 16?

@colebemis
Copy link
Contributor

This icon is originally 12px. Should we make it in 24 instead of 16?

No, we should still use the 16px grid.

@colebemis
Copy link
Contributor

colebemis commented Jan 29, 2020

“Ellipses” has a different meaning than the similar “Kebab” icon so we should probably keep them visually different. According to our interface guidelines, “Ellipses” is for toggling truncated inline text content. “Kebab” is for toggling inline dropdowns and menus. https://primer.style/design/ui-patterns/progressive-disclosure#progressive-disclosure

V1 and V2 might be a little too similar to the “Kebab” icon unless we want to change “Ellipses” to just be a smaller “Kebab” icon.

@ashygee ashygee added must and removed todo: GA labels Jan 29, 2020
@edokoa
Copy link
Contributor

edokoa commented Jan 29, 2020

Then I'm fine with keeping the black box version.

@colebemis
Copy link
Contributor

This is isn't being used in any public facing UI so I think we can remove the must label.

@edokoa
Copy link
Contributor

edokoa commented May 27, 2020

Would we go with this one then?
https://www.figma.com/file/2rjsQp42uKvzPXzLZIqOeA/Octicons-v2-Source?node-id=1109%3A3789

Do we need a 24px version?

image

@ashygee
Copy link
Contributor Author

ashygee commented May 28, 2020

I think I prefer what you had going on here @edokoa
image

@edokoa
Copy link
Contributor

edokoa commented May 29, 2020

With and without border here

https://www.figma.com/file/2rjsQp42uKvzPXzLZIqOeA/Octicons-v2-Source?node-id=1119%3A3282

The solid one is closer to the current one, maybe the border one is complex for a small size. Would we need a 24px?

image

@colebemis
Copy link
Contributor

I like the solid one ❤️ The outline one feels cluttered and blurry and I don't think there is a need for it. I'm okay with only having a solid version of this icon. Should we call it ellipsis-fill or just ellipsis?

Re: 24px — This icon is meant to be used inline with text so I can't imagine a 24px use for it.

@colebemis
Copy link
Contributor

I opened a PR to add this to Octicons: primer/octicons#435

I'll merge when I have approvals from @ashygee and @edokoa

@ashygee ashygee closed this as completed Jun 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants