-
Notifications
You must be signed in to change notification settings - Fork 827
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
Octicon request: Archived repo #182
Comments
I'll grab this one, it's a good excuse to go through the entire flow (and document it) with the new monorepo. |
Hey, hope it's alright to jump in on this issue, but I spoke with @jonrohan and @broccolini yesterday about also needing an "archive" icon, and we decided I could be the guinea pig for the new octicon contributor flow via Figma. So here we go! I've added 2 versions of an archive icon in this file. One is based on some of the feedback from this issue linked above, and the other is an alternative I thought could be interesting, and holds up a little better at a smaller scale. Link to my draft of the octicons file: https://www.figma.com/file/taeondve1a0gWuNuEN4RJ3/Octicons-Colin-Adding-Archive-Icon Screenshots: In context (issue timeline) |
Yeah I had similar feelings about how the outlined box icon held up in comparison to the arrow icon. I'm leaning a little more in the direction of the box icon only for the sake of clarity. I do think the arrow icon has some traction as an "archive" icon, but I don't have examples right now to back that up. |
Bump! We're making good progress on the project card archival feature and I would love to be able to get an icon sorted out for this so we can start including it in the UI. What can I do to help move this along? |
@colinkeany I think the original outline box works best and fits with the other icons. |
@broccolini yup that's good with me! |
We need an icon to represent an archived repo. No major rush on it, but if/when we start adding icons again or redesign the system, I wanted to record this.
UI examples and conversation happening here: https://github.com/github/github/pull/80421)
The text was updated successfully, but these errors were encountered: