divyanshudhruv / Minifolio

⚡ A minimal portfolio template for Developers | 2024
https://divyanshudhruv.github.io/Minifolio/
MIT License
36 stars 40 forks source link

feat: Add dark mode #2 #47

Open rahulpeacock opened 11 months ago

rahulpeacock commented 11 months ago

Completed the feature Dark mode, issue - #2

divyanshudhruv commented 11 months ago

@Rahul-Palamarthi can you share a pic ?

rahulpeacock commented 11 months ago

@divyanshudhruv yes I will share the pic

divyanshudhruv commented 11 months ago

Ok please

rahulpeacock commented 11 months ago

@Rahul-Palamarthi can you share a pic ?

@divyanshudhruv here is the vid of the dark-mode

https://github.com/divyanshudhruv/Minifolio/assets/87606332/81d5bb30-d856-4a90-a955-e2a7502707da

divyanshudhruv commented 11 months ago

Cool

divyanshudhruv commented 11 months ago

Will merge after 6 oct

divyanshudhruv commented 11 months ago

Is the button responsive?

rahulpeacock commented 11 months ago

Is the button responsive?

The size of the button remains same (fine for mobile view also)

divyanshudhruv commented 11 months ago

Idk why but the terminal typing text is not visible in dark mode

rahulpeacock commented 11 months ago

I can fix it

divyanshudhruv commented 11 months ago

If you deploy your forked repo, it would be very easy for me to view changes. So please deploy the changed branch

rahulpeacock commented 11 months ago

Idk why but the terminal typing text is not visible in dark mode

I have fixed it and also made a PR

https://github.com/divyanshudhruv/Minifolio/assets/87606332/86bc50de-fd9c-4325-bf1a-405b14825ed8

rahulpeacock commented 11 months ago

If you deploy your forked repo, it would be very easy for me to view changes. So please deploy the changed branch

@divyanshudhruv should I deploy to gh-pages ?

divyanshudhruv commented 11 months ago

Yes yes, then share me the link, I will see then merge

divyanshudhruv commented 11 months ago

I think you should remove Your Name from the header.

rahulpeacock commented 11 months ago

I think you should remove Your Name from the header.

i mean there should be a placeholder so that the header styles will work properly (since using flex inside the header so that the dark-mode button will be at the end of the header)

rahulpeacock commented 11 months ago

Yes yes, then share me the link, I will see then merge

yes I will share the link

divyanshudhruv commented 11 months ago

Yes yes, then share me the link, I will see then merge

yes I will share the link

Ok, fast

rahulpeacock commented 11 months ago

Yes yes, then share me the link, I will see then merge

yes I will share the link

Ok, fast

@divyanshudhruv, Here is the link to the gh-pages from my forked repo

divyanshudhruv commented 11 months ago

Hello, please resolve some conflicts in the files

rahulpeacock commented 11 months ago

Hello, please resolve some conflicts in the files

Yes, I have solved the merge conflicts 🚀

divyanshudhruv commented 11 months ago

Can you remove the border from the button and set the button background as #f0f0f0 ? And also increase the size some.

rahulpeacock commented 11 months ago

Yes

divyanshudhruv commented 11 months ago

Ok please do

rahulpeacock commented 11 months ago

Ok please do

@divyanshudhruv, I have update the code and made a PR, You can see the changes in the gh-pages of my forked repo

divyanshudhruv commented 11 months ago

Ok, can I do some minor fixes in the code ? Then I will merge it

rahulpeacock commented 11 months ago

Ok, can I do some minor fixes in the code ? Then I will merge it

@divyanshudhruv I will be working on it 🍁

rahulpeacock commented 11 months ago

Ok, can I do some minor fixes in the code ? Then I will merge it

@divyanshudhruv I have completed the merge conflicts 🚀

divyanshudhruv commented 11 months ago

Ok so see the current portfolio and there is a back to top button. Just make the dark button as same like that. I mean the width,height the background color, the font color. And the hover style you can do on your own.

rahulpeacock commented 11 months ago

Ok so see the current portfolio and there is a back to top button. Just make the dark button as same like that. I mean the width,height the background color, the font color. And the hover style you can do on your own.

sure

divyanshudhruv commented 11 months ago

@Rahul-Palamarthi please resolve the conflicts

rahulpeacock commented 11 months ago

@divyanshudhruv sure

divyanshudhruv commented 10 months ago

Added the hacktoberfest-accepted label, will merge soon