Skip to content
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

Progress bar background color #3307

Open
Le0X8 opened this issue Oct 1, 2023 · 33 comments · May be fixed by #3312
Open

Progress bar background color #3307

Le0X8 opened this issue Oct 1, 2023 · 33 comments · May be fixed by #3312
Assignees
Labels
enhancement New feature or request. feature hacktoberfest lang-card Issues related to the language card.

Comments

@Le0X8
Copy link

Le0X8 commented Oct 1, 2023

Is your feature request related to a problem? Please describe.

I want to use the top-langs badge with the transparent theme, but I can't find an option to set the progress bar background color.

Screenshot 2023-10-01 at 13-08-40 Le0X8 (LeoX DEV)

Describe the solution you'd like

I would like an additional option to set the background color to make the badge look like in this image (edited with GIMP):
Screenshot 2023-10-01 at 13-08-40 Le0X8 (LeoX DEV) (better)

The additional option could be called prog_bg_color, so my example would be something like: https://github-readme-stats.vercel.app/api/top-langs/?username=Le0X8&size_weight=0.5&count_weight=0.5&langs_count=8&theme=transparent&border_radius=0&hide_border=true&title_color=108c92&text_color=0c686c&prog_bg_color=222222.

Describe alternatives you've considered

No response

Additional context

No response

@Le0X8 Le0X8 added the enhancement New feature or request. label Oct 1, 2023
@JyothsnaVellampalli
Copy link

I wan to work on this. Shall I start?

@Le0X8
Copy link
Author

Le0X8 commented Oct 1, 2023

@JyothsnaVellampalli sure why not

a-s-t-e-y-a added a commit to a-s-t-e-y-a/github-readme-stats that referenced this issue Oct 1, 2023
@a-s-t-e-y-a
Copy link

a-s-t-e-y-a commented Oct 1, 2023

I have enhanced the code by adding the prog_bg_color feature. Please review the pull request and consider merging it if it meets your requirements. If it doesn't, please provide feedback or state the reasons for not merging it.

My Pr : - #3312

@a-s-t-e-y-a
Copy link

Could you please review my pull request, @anuraghazra ?

@a-s-t-e-y-a
Copy link

Could you please review my pull request, @rickstaa ?

@Pratik1603
Copy link

please assign this to me

@abhi5hek001
Copy link

could you please assign this task to me?
i would love to solve this issue @anuraghazra

@doshinxlsx
Copy link

Hey @anuraghazra , pleasure to meet you! I created a new pr, would be happy if you could check and approve it if it meets your requirements. If you have anything related to React, Javascript, Typescript or Python, feel free to hit me up!
Thanks in advance!

@qwerty541
Copy link
Collaborator

I have enhanced the code by adding the prog_bg_color feature. Please review the pull request and consider merging it if it meets your requirements. If it doesn't, please provide feedback or state the reasons for not merging it.

My Pr : - #3312

@a-s-t-e-y-a I left my review, check it out #3312 (review)

@qwerty541 qwerty541 added lang-card Issues related to the language card. feature labels Oct 11, 2023
@airwakz
Copy link

airwakz commented Oct 15, 2023

@qwerty541 can I do this task

@airwakz
Copy link

airwakz commented Oct 15, 2023

@rickstaa i think that guy is hasn't contributed for last 4 days so can uh pass on this task to me

@qwerty541
Copy link
Collaborator

@qwerty541 can I do this task

Hey, @airwakz! Thanks for your interest in our project! We already have a pull request which will solve this issue #3312. I recently left my review with requested changes, in case if @a-s-t-e-y-a do not have enought time to do it, may be you can finish this pull request in co-authored mode. Let's wait for @a-s-t-e-y-a's response. Also feel free to choose another issue that you are interested to work on and contact me about it.

@airwakz
Copy link

airwakz commented Oct 15, 2023

ok thanks

@airwakz
Copy link

airwakz commented Oct 16, 2023

@qwerty541 can i make the required changes now

@rickstaa
Copy link
Collaborator

rickstaa commented Oct 16, 2023

@qwerty541 can i make the required changes now

@airwakz another user already implemented this feature in #3312 but some changes are still required.

@qwerty541
Copy link
Collaborator

@airwakz I'm okay if you finish this pull request in co-authored mode, but i do not see an easy way to do it because you don't have write access to @a-s-t-e-y-a's branch.

@airwakz
Copy link

airwakz commented Oct 17, 2023

@qwerty541 I have a way but it is quite complex I will make changes in my report as per pull req and than I will finish the review asked by uh

@qwerty541
Copy link
Collaborator

@qwerty541 I have a way but it is quite complex I will make changes in my report as per pull req and than I will finish the review asked by uh

@airwakz I think that the best way will be the following:

  1. You should add @a-s-t-e-y-a's fork as remote repository git remote add a-s-t-e-y-a git@github.com:a-s-t-e-y-a/github-readme-stats.git
  2. Fetch it git fetch a-s-t-e-y-a
  3. Create a new local branch based on the one from which feature(top langs card): add ability to change progress bar background color in normal layout (resolves #3307) #3312 was created.
  4. Make changes and push new branch into your own fork.
  5. Create a pull request in @a-s-t-e-y-a's fork on the correct branch.
  6. I hope i will be able to approve and merge it, or if not we should wait until @a-s-t-e-y-a will do this.

@qwerty541
Copy link
Collaborator

@rickstaa Have you already experienced when users worked on issue together? Do you have a better solution?

@airwakz
Copy link

airwakz commented Oct 17, 2023

image
error shown

@qwerty541
Copy link
Collaborator

@airwakz If you didn't configured own SSH key for GitHub you can try use HTTPS instead: git remote add https://github.com/a-s-t-e-y-a/github-readme-stats.git

@airwakz
Copy link

airwakz commented Oct 17, 2023

@qwerty541 I just did that and it work thanks for the solution

@airwakz
Copy link

airwakz commented Oct 17, 2023

@qwerty541 i have used git clone and imported into my local machine can uh tell me the steps to commit here

@airwakz
Copy link

airwakz commented Oct 17, 2023

@qwerty541 actually i encountered another issue right now again
so i have cloned it now i have to make a branch in a github so that it doesnt affect my alerady done pr

@rickstaa
Copy link
Collaborator

@qwerty541 I have a way but it is quite complex I will make changes in my report as per pull req and than I will finish the review asked by uh

@airwakz I think that the best way will be the following:

  1. You should add @a-s-t-e-y-a's fork as remote repository git remote add a-s-t-e-y-a git@github.com:a-s-t-e-y-a/github-readme-stats.git
  2. Fetch it git fetch a-s-t-e-y-a
  3. Create a new local branch based on the one from which feature(top langs card): add ability to change progress bar background color in normal layout (resolves #3307) #3312 was created.
  4. Make changes and push new branch into your own fork.
  5. Create a pull request in @a-s-t-e-y-a's fork on the correct branch.
  6. I hope i will be able to approve and merge it, or if not we should wait until @a-s-t-e-y-a will do this.

This is what I would do in your case as well. This way you can open a new pull request in which the commits by @a-s-t-e-y-a are included. As a collaborator (i.e. maintainer), I or @qwerty541 can even rebase your changes on #3312 if we want 👍🏻.

@rickstaa
Copy link
Collaborator

rickstaa commented Oct 17, 2023

@qwerty541 actually i encountered another issue right now again so i have cloned it now i have to make a branch in a github so that it doesnt affect my alerady done pr

It would be best if you always worked from a feature branch. For more tips on how to contribute to OS projects, please see https://opensource.guide/.

@airwakz
Copy link

airwakz commented Oct 17, 2023

image
so this work onky for maintainer

@rickstaa
Copy link
Collaborator

image so this work onky for maintainer

No, since GRS is public @a-s-t-e-y-a's fork also is public, you should be able to add the fork as a remote and fetch all references. When people select this option, the only thing available for maintainers is pushing to people's forks directly. People can enable or disable this on their pull request.

image

I performed some tests and the following works when not being a maintainer:

  1. git clone git@github.com:airwakz/github-readme-stats.git
  2. git remote add asteya git@github.com:a-s-t-e-y-a/github-readme-stats.git
  3. git fetch asteya
  4. git checkout asteya/master - This will give you a You are in 'detached HEAD' state warning.
  5. git switch -c progress_background - To fix this, you must create a new branch.

After these steps, you will have a progress_background branch with all the changes of @a-s-t-e-y-a included and you can work from there.

A guide on how this works can be found here. The difference is that you and @a-s-t-e-y-a work from the master branch instead of a feature branch. As a result, an extra step is required since your master conflicts with their master 😅. Because of this, I always advice people to use a feature branch.

@airwakz
Copy link

airwakz commented Oct 18, 2023

@qwerty541 can uh assign me this i will make a pr soon today with the changes as uh requested

@qwerty541
Copy link
Collaborator

@qwerty541 can uh assign me this i will make a pr soon today with the changes as uh requested

@airwakz I left my review, check it out #3388 (review)

@qwerty541
Copy link
Collaborator

image so this work onky for maintainer

I also recommend you to configure SSH key on your GitHub account https://docs.github.com/en/authentication/connecting-to-github-with-ssh/adding-a-new-ssh-key-to-your-github-account?tool=webui
Until you perform it you should use HTTPS link https://github.com/a-s-t-e-y-a/github-readme-stats.git instead of SSH git@github.com:a-s-t-e-y-a/github-readme-stats.git

@Le0X8
Copy link
Author

Le0X8 commented Apr 10, 2024

Why does nobody work on this?

PR #3312 seems to be abandoned, and all the other pull requests have been closed in favor of #3312.

@qwerty541
Copy link
Collaborator

I left my review #3312 (review) with changes request a long time ago, I'm almost always here to review the pull request If the changes will be made. Also I'm okay with replacing #3312 if someone will open another pull request with all required changes.

However, according to our team rules, merging features require 2+ collaborators approval, at the moment I'm alone active, therefore even if the pull request will be ready and approved by me, merging will be postponed indefinitely until any team member will be free.

As you see there is no new features last 4-5 months, just maintenance routine like dependencies update, fixing vulnerabilities, docs typos, answering issues and discussions etc. I hope on your understanding and patience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request. feature hacktoberfest lang-card Issues related to the language card.
Projects
None yet
10 participants