Skip to content

/ atom

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

Jump to bottom

Update Octicons to v10.0.0 #21049

Open
johnletey opened this issue Jul 7, 2020 · 2 comments
Open

Update Octicons to v10.0.0 #21049

johnletey opened this issue Jul 7, 2020 · 2 comments
Labels
good first issue hacktoberfest

Comments

@johnletey
Copy link

@johnletey johnletey commented Jul 7, 2020

Summary

GitHub has recently released some new snazzy octicons, and would love to see them in Atom!

Motivation

To look more cutting edge 😍

Describe alternatives you've considered

Additional context
@gopipofficial
Copy link

@gopipofficial gopipofficial commented Oct 4, 2020 •
edited

I would like to work on this as part of Hactoberfest and to start my experience on open source. I'm working on Atom local setup and can you kindly explain to me what to achieve in this issue?. that would be helpful for me to get a start.
@sadick254
Copy link
Contributor

@sadick254 sadick254 commented Oct 5, 2020

Hey @gopipofficial the best place to start would be

atom/static/variables/octicon-mixins.less

Lines 36 to 48 in 0f29759

.octicon-font-legacy() {
font-family: 'Octicons Regular';
src: url("icons/octicons-2.1.2.woff") format("woff");
font-weight: normal;
font-style: normal;
}
.octicon-font() {
font-family: 'Octicons Regular';
src: url("icons/octicons.woff") format("woff");
font-weight: normal;
font-style: normal;
}

However looking at the direction that github took to deliver octicons as svg rather than icon fonts, working on this would require a rework on how atom uses the icons.

Apologies I miss labeled this issue. If you are still willing to proceed working on this issue let me know.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue hacktoberfest
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@sadick254 @gopipofficial @johnletey
You can’t perform that action at this time.