-
Notifications
You must be signed in to change notification settings - Fork 2
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
Enhance looks #5
Comments
Hi, I am new to open source and am interested in helping. I was wondering if I could take this on and if you have any idea the best way to get started? |
Hi Alex! Yeah, I'm always open to ideas. Feel free to clone the repo! 👍 As a start, create a sample react project, or use an existing one, and use 'npm link' to add this component as a dependancy. That way you can see changes in real-time. Do you have any experience with React? |
Okay. Yes, I've had some limited experience but I'm definitely interested in learning more! I noticed you have some other issues as well. Maybe I could try and work on some other ones after I complete this one as well. I will get started on this right away. |
Great! Yeah, I made some issues to keep track of enhancements I'd like to get done, as a reminder to myself, but I'd appreciate some input. 👍 Let me know if you have any questions! |
@AlexErling I added a small guide to get you started. Let me know if you have any questions! |
I'm having a little trouble using npm link. So far I used npm link in the react-github-showcase, and then npm link react-github-showcase in my sample app. But I'm still having issues getting it to work.
but that doesn't work. This is what I've put this in the app.js on my sample project.
Any suggestions? |
Okay, I figured out what I was doing wrong. I needed to do npm start after the link. But now I am getting this error.
|
Yes, that was an error that should be fixed now if you do a pull from the master branch. Also, you can get your api-key from here |
Nice! Glad you got it to work. In my project, I have it wrapped in another component that provides styling, as well as having defined styles in general. The Github Showcase component does not come with any inherent styles by itself. The component I defined in my project was called CardPage which had it's own stylesheet. Feel free to have a look through my code! I build my project using Gatsby and used Typography to define some basic styling. |
Okay thanks for the info, so you're trying to keep the styling on the showcase component to a minimum? Just curious how much I should change. I changed the commit date to appear grey. What else would you like me to do then? Any ideas on what you would want to do to the commit id tag. |
Yeah, that was the idea. If a theme changes the fonts and colors of text depending on background color, then we want this component to respond. So ideally it's a matter of changing the opacity of the commit-date instead of changing color, if that makes sense. Still unsure of commit tag. Could be cool to have a mini-photo or name to indicate who made the change! |
* Darken commit-date as mentioned in issue #5
The text was updated successfully, but these errors were encountered: