Changing theme of Most Used Languages

[Safuan12616]

Changing theme of Most Used Languages.

You can also change themes of Most Used Languages.
You just need to change the theme name
&layout=compact&theme=theme name
Preview:

Here I used discord_old_burple theme.

[anuraghazra]

Yeah it's documented: https://github.com/anuraghazra/github-readme-stats#common-options

[modz2014]

I have noticed that it's not even if you look here the yellow ones for Pascal are higher then the others and it's a noticeable difference

[qwerty541]

Hey, @modz2014! I'm not sure I understood you correctly, but if you're talking about languages colors, then we don't define them ourselves. We load them from https://github.com/github-linguist/linguist using special workflow update-langs.yaml. It means that our card shows same languages color that GitHub interface.

[modz2014]

Upon careful examination of the image, it becomes evident that the elements do not align in a cohesive manner, resulting in an overall appearance of irregularity and imbalance.

[qwerty541]

I suppose that you're talking about different length of strings with programming languages names and percentages. If you don't like this layout feel free to propose the better one. We already provide several different layouts for top languages card.

[modz2014]

I like this one if you look closer in the details you will notice but it does not show the defect when your on a phone which is werid

…

On Mon, 6 Nov 2023, 3:05 am Alexandr Garbuzov, ***@***.***> wrote: I suppose that you're talking about different length of strings with programming languages names and percentages. If you don't like this layout feel free to propose the better one. We already provide several different layouts <https://github.com/anuraghazra/github-readme-stats#demo-2> for top languages card. — Reply to this email directly, view it on GitHub <#1329 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGCKZY3XJEZPYJVBKMKQXCLYC7PPTAVCNFSM5EZVQQ5KU5DIOJSWCZC7NNSXTOKENFZWG5LTONUW63SDN5WW2ZLOOQ5TONBYGA4TSMQ> . You are receiving this because you were mentioned.Message ID: <anuraghazra/github-readme-stats/repo-discussions/1329/comments/7480992@ github.com>

[qwerty541]

It would be great if you could attach a screenshot of the defect and circle it for clarity, since apparently all this time we are talking about different things.

[modz2014]

i did above mate

[qwerty541]

I see this screenshot #1329 (comment) but i don't see any defect on it, so i asked you to circle it.

[modz2014]

itrs only small detials but it works fine when i view it on a mobile phone

[qwerty541]

Yes, I see this little piece where two colors overlap each other. I checked the code which generates compact layout of top languages card and noticed that each next part of progress bar starts from same coordinates where previous ends, without any indent. https://github.com/anuraghazra/github-readme-stats/blob/master/src/cards/top-languages-card.js#L360-L382

In general, this is not critical, but of course it can be fixed, but you will need to very carefully select the size of the indentation so that the black defect does not change to white.

@rickstaa What do you think? Is it worth to change this in your opinion?

[modz2014]

would make it look nicer though

[rickstaa]

Yes, I see this little piece where two colors overlap each other. I checked the code which generates compact layout of top languages card and noticed that each next part of progress bar starts from same coordinates where previous ends, without any indent. https://github.com/anuraghazra/github-readme-stats/blob/master/src/cards/top-languages-card.js#L360-L382

In general, this is not critical, but of course it can be fixed, but you will need to very carefully select the size of the indentation so that the black defect does not change to white.

@rickstaa What do you think? Is it worth to change this in your opinion?

Since this is a minor bug, it should be low on our priority list. I didn't notice it until you brought it to my attention 😅 . Having that said, maybe @modz2014, you can create a bug report for this, and if people want, they can then create a pull request we can review 👍🏻.

[qwerty541]

I would like to solve this problem personally, but unfortunately it is not reproducible on any of my devices. Apparently its appearance also depends on the screen resolution, or something similar. This behavior once again casts doubt on making any fix for this at all.

[modz2014]

So it works fine on my phone i see must but the screen i use seems werid

…

On Mon, 13 Nov 2023, 4:39 am Alexandr Garbuzov, ***@***.***> wrote: I would like to solve this problem personally, but unfortunately it is not reproducible on any of my devices. Apparently its appearance also depends on the screen resolution, or something similar. This behavior once again casts doubt on making any fix for this at all. [image: image] <https://user-images.githubusercontent.com/53787217/282324208-2a1d7a27-06c4-433b-8c0c-bfb1d79cb133.png> — Reply to this email directly, view it on GitHub <#1329 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGCKZY46I2JKXQVBFZCCDY3YEEXWVAVCNFSM5EZVQQ5KU5DIOJSWCZC7NNSXTOKENFZWG5LTONUW63SDN5WW2ZLOOQ5TONJUG44DENA> . You are receiving this because you were mentioned.Message ID: <anuraghazra/github-readme-stats/repo-discussions/1329/comments/7547824@ github.com>

[rickstaa]

Hey @qwerty541, I'm on board with the idea of tracking this issue, but I reckon #3469 could use a bit of a makeover. A clear example, some visuals, and a more polished problem description would work wonders, don't you think?

[qwerty541]

@rickstaa Yes, I completely agree with you that this issue needs to be redone. Let's give @modz2014 ability to make the suggested changes since this issue was opened by him. Of course we can provide him help later if he needs.

[modz2014]

i have to look into more but im sure you guys know :)

[qwerty541]

@rickstaa I have updated this issue #3469 and added my comment. Please share your minds about if it become clear now or not.

[rickstaa]

@qwerty541 ah, great! Thanks a lot for putting in the effort to update the issue 🚀. Let's use the issue to see how many people are affected 👍🏻.