Added Github Stats and Most Used Languages#41
Conversation
|
✔️ Deploy Preview for practically-pi-portfolio-template ready! 🔨 Explore the source changes: d9de09a 🔍 Inspect the deploy log: https://app.netlify.com/sites/practically-pi-portfolio-template/deploys/60f876bdf79e950008c3f430 😎 Browse the preview: https://deploy-preview-41--practically-pi-portfolio-template.netlify.app/ |
louisefindlay23
left a comment
There was a problem hiding this comment.
Awesome job, @Inoxia25. Looks great. Love the most used languages card you added. On smaller screens, the cards are bunched together, so it would be good to add a margin in a media query to account for this. See image below.
If you wanted to extend it, maybe try and style the Github Stats card to match the colour scheme of the portfolio ?
|
@louisefindlay23 hey changed the color scheme for the github stat card |
|
Awesome work on the colour scheme. Still see the margin issue. Will chat further in our 1 on 1. |
louisefindlay23
left a comment
There was a problem hiding this comment.
That should be it resolved. :) Needed to be .github-stats img instead of the container class. Lovely conversation. Great to hear about your experiences and was great to chat about uni and hackathons.
|
@louisefindlay23 yup it was so great talking to you! Thank you for all the recommendations! |
|
Should be an easy fix. Will take a look after standup |
|
Hmm, I can't look because I don't have write access or are PR author but basically install Refined Github then go to Extension Settings and generate a Github token, paste it into the field and then go to resolve conflicts in the web editor. |
|
May do my talk on Git Merge Conflicts since it seems to be a topic of high interest and hopefully be very useful unless I learn some new and interesting new thing before then. Learned about ternary operators in JS but not sure if I can make a 10 min talk about it since I haven't had the change to use them yet. |
|
@Inoxia25 looks sharp. For the dark mode toggler, I think we should look into modifying the Vercel URL from I'll wait until the PR gets merged and then try come up with a scrip to modify the url (feel free to try it too if you want) |
|
@louisefindlay23 yes sounds like a plan! Who else to learn about git merge from than the git princess herself! 👑 |
|
@dtemir yup would look great! there are actually many themes available for the github stat card, so once we decide the color scheme for the dark mode, we can choose accordingly 👀 |
Why This PR Adds Value
This PR gives more idea about the users contributions and most used languages on Github
What This PR Adds
Screenshot
Issue This PR Closes
This closes #29