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

Fix: blog page from "Blog.jsx" #155

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

AryanKuAg
Copy link
Contributor

Hi @ItsRoy69 , I've fixed this issue: #91
Pls merge my code🙂 and could u pls make me a contributor of this project and add my profile in the project's readme.md file, nobody has ever added me, pls, it will make my day🙏
image

Copy link

vercel bot commented Jan 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
doodle-collab ❌ Failed (Inspect) Feb 11, 2024 9:06am

@ItsRoy69
Copy link
Collaborator

@AryanKuAg needs few checks

Tailwind is used for eliminating the need to write custom CSS for many common styles, thus proper readibility is required by breaking down the styles into modular components and then create a separate CSS file for different components or sections.

.btn { @apply px-4 py-2 rounded-md bg-blue-500 text-white; }

Here, the cards are breaking and will suggest to work on the blogs section first, and modify few changes like giving the same background color as given on homepage and features page and then start working on tailwindcss

image

@AryanKuAg
Copy link
Contributor Author

Okay sure @ItsRoy69 . I'll start this from tmrw onwards cause today's my office schedule will gonna be a little hectic.

@ItsRoy69
Copy link
Collaborator

Okay sure @ItsRoy69 . I'll start this from tmrw onwards cause today's my office schedule will gonna be a little hectic.

no issues..chill

@ItsRoy69
Copy link
Collaborator

ItsRoy69 commented Feb 1, 2024

@AryanKuAg pushed blogs changes

@AryanKuAg
Copy link
Contributor Author

image
image
image
Hi @ItsRoy69 , I've fixed the issues with the card suggested by u and made it responsive. Can I start working on the tailwind conversion? and pls merge this PR.

@AryanKuAg
Copy link
Contributor Author

image
Please let me know, How can I add myself here? @ItsRoy69

@AryanKuAg
Copy link
Contributor Author

And also Pls unassigned me this task cause I will not be available for 1 week, after then I can continue. @ItsRoy69

@ItsRoy69
Copy link
Collaborator

ItsRoy69 commented Feb 1, 2024

@AryanKuAg thanks for informing, if you're okay to work again then i'll keep this issue on hold till 10th February, 2024 as there is more work to be done and it is time consuming and you can join the discord and tag me for the further discussion https://discord.com/invite/Ar84xkXkZt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants