-
Notifications
You must be signed in to change notification settings - Fork 996
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
Made a Frontend for repo using GitHub API, React and Chakra UI. #358
Comments
Todos:
I won't be too active on this project but I'll try to keep working, I have some exams next month so need to study 😔 |
Thanks for letting me know @kotx :) I'm unsure about why the first bug is happening, but maybe it's because of chakra ui's global styles, I'll try to fix it today only. About scrollbars, I'm planning to style them something like this: About the third one, I was thinking of this even myself, I'm thinking of just making it load images of a particular language by default, will this work? |
@cat-milk |
The question I would ask is would it be worth the hassle/effort though? When you say "add their projects", can you please elaborate? |
Scrollbar looks good 👍 |
What I'm trying to say is having an organization will be much more cleaner and organized rather than trying to merge them in the main repo or keeping them separate, Not so sure if this is a good idea, but can you explain what do you mean by "worth the hassle/effort"? |
What I meant by my comment is what your proposing worth it in the long run? |
I don't see a significant difference in people owning their own repos as opposed to some org owning it. |
people would be able to find all the projects related to this repo in a single place? 🤔 |
What's wrong with a list of them in the README? |
nothing's wrong with it, this could work too. |
@kotx |
So I guess I have fixed almost everything now, List of changes I made:
|
Yup, looks much better, thanks! Sorry to bug you again, but the image viewer's icons don't seem to be aligned with the image title and slightly offscreen: Overall looks good though, thanks for making this! |
Yes, their are 2 scrollbars actually, one is for body and another is for the grid, because the grid was overflowing without adding the scroll. Thanks :) |
@kotx |
This is epic. |
React is shit lol |
Oh dude you fuckin' gottem bro. |
better than using vanilla js for me |
@marshallovski Tf you're talking about, vanilla is trash if you're building a website, you'll end up making your own shitty javascript framework if you use vanilla lol. |
LGTM |
React nowadays is pretty fast, you can make it as faster as vanilla Javascript most of the time. Not only React, pretty much all of the modern frameworks such as vue and svelte are pretty fast as well. I really don't see why people hate React so much. |
I'm new to frontend dev, so don't mind the design, made it in around 8 hours. Would definitely improve the UI, contributions are most welcome :)
Site: https://waifus-for-programmers.vercel.app/
Repo Link: https://github.com/asheeeshh/waifus-for-programmers
edit: repo was rewrote using nextjs and tailwind css
Please let me know where can I improve.
![image](https://user-images.githubusercontent.com/68690233/156751723-b4d48cc9-9bfc-4ff3-96f5-7a47f6fe523d.png)
The text was updated successfully, but these errors were encountered: