afrog icon indicating copy to clipboard operation
afrog copied to clipboard

Update Octicons to v10.0.0

Open johnletey opened this issue 4 years ago • 42 comments

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

johnletey avatar Jul 07 '20 10:07 johnletey

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.

iam-gopi avatar Oct 04 '20 09:10 iam-gopi

Hey @gopipofficial the best place to start would be https://github.com/atom/atom/blob/0f2975930e506785c05068a1c57934160e094e10/static/variables/octicon-mixins.less#L36-L48

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.

sadick254 avatar Oct 05 '20 07:10 sadick254

I'll try my level best to achieve this before to start I'm having local setup issue working on it. Please give me sometime.

iam-gopi avatar Oct 07 '20 02:10 iam-gopi

hi @gopipofficial are you working on this ?

pavan5555 avatar Oct 08 '20 01:10 pavan5555

Yes, @pavan5555 ill be working on it. Unfortunately, I'm facing an atom local setup issue. Once I fixed that ill start working on it.

iam-gopi avatar Oct 08 '20 02:10 iam-gopi

@sadick254 I regret to say this, I'm still facing an issue with the local setup. I've opened one issue for getting help on that.

iam-gopi avatar Oct 09 '20 03:10 iam-gopi

Hi, @gopipofficial . Are you still working on this?

Moe82 avatar Oct 21 '20 23:10 Moe82

@Moe82 my local environment setup issue was fixed with the help of @sadick254 and now I started looking into the issue.

iam-gopi avatar Oct 22 '20 04:10 iam-gopi

@sadick254 thanks for your input on setting up the local env. now its up and wanted to fix this issue. the issue is to change all the icons in the Atom from font icon to new octicons respectively right ???

iam-gopi avatar Oct 22 '20 05:10 iam-gopi

Thats correct @gopipofficial

sadick254 avatar Oct 22 '20 10:10 sadick254

Thanks, I'm starting with the changes.

iam-gopi avatar Oct 23 '20 03:10 iam-gopi

Wow that was so help help full for me thnaks for sharing such a valuable information i use these tips on my website SolankiGaming! and I am able to generate a lot of traffic.

Solankigaming avatar Dec 19 '20 16:12 Solankigaming

It's very informative. I really appreciate your works for the newbies. I follow these on my Blog HindiDigit

hindidigit avatar Feb 15 '21 09:02 hindidigit

i would like to work on it plz explain what exactly i have to do

subhamagarwal624 avatar Feb 20 '21 09:02 subhamagarwal624

Hey if this issue is still open I would like to work on it.

nidhishtyagi avatar Apr 09 '21 08:04 nidhishtyagi

Thank you so much for this article. it was very informative. i really hope you continue making great content like this. i will definently follow these tips on my blog: Gadgetsglam

angelstranger1111 avatar Apr 30 '21 06:04 angelstranger1111

This issue is still open

icecream17 avatar Apr 30 '21 12:04 icecream17

Is this issue still open I'd like to contribute?

cloudguruab avatar May 16 '21 01:05 cloudguruab

@cloudguruab this won't be easy. Octicons quit using fonts in v5 and went with svgs. But you can try if you want to.

UziTech avatar May 16 '21 02:05 UziTech

@UziTech I will give it a go

cloudguruab avatar May 16 '21 02:05 cloudguruab

I found 2 resource in case it helps:

https://docs.brainstormforce.com/how-to-convert-svg-file-into-a-font-icon/

https://stackoverflow.com/questions/13278707/how-to-convert-svg-files-to-a-font/

ThatXliner avatar May 16 '21 02:05 ThatXliner

@ThatXliner Thank you!

cloudguruab avatar May 16 '21 20:05 cloudguruab

@BIGBANGDEVTRON Shopee_695a45941defa7ced2dfce1f9120ed77_1

BIGBANGDEVTRON avatar May 17 '21 12:05 BIGBANGDEVTRON

@sadick254 @UziTech Good job and thanks!!!

I hope those octicons...

deCaldas avatar Jun 05 '21 15:06 deCaldas

Would performance have a hit if we instead replaced the spans with SVGs?

ThatXliner avatar Jun 06 '21 23:06 ThatXliner

@parrthpanchal go ahead 👍

UziTech avatar Jun 19 '21 16:06 UziTech

Looks like it is an error in https://github.com/atom/atom/blob/978843584aaeb25e25fbe53693bb9d227b4bfed1/script/lib/spawn-sync.js#L17

If you delete ${result.args.join(' ')} locally it should not throw an error.

UziTech avatar Jun 21 '21 21:06 UziTech

WHEN WILL YOU FUCKING STOP YOUR SPAMS, HERE, ALL OF YOU? I UNSUSCRIBED MILLION TIMES AND STILL RECIEVING YOUR SHIT !

LAST ADVISE : OR YOU ALL STOP TO SPAM MY FUC.KING MAILBOX WITH ALL YOUR SHIT, OR I FUCKING BRING YOU TO COURT !

ENOUGHT CLEAR ?

metafusion3D avatar Jun 21 '21 22:06 metafusion3D

@parrthpanchal yes

UziTech avatar Jun 22 '21 02:06 UziTech