-
Notifications
You must be signed in to change notification settings - Fork 112
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
Updated Volunteers section with Grids #100
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you've changed anything cuz I am seeing this, @davyj123
Done. |
that's my little mistake |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eview.mp4
ok mam |
mam actually working proper at my local repo |
Updated succesfully |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix these @davyj123 :
Done |
@2024-SANDHYA done with all changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@davyj123 This is original one :
This is where you messed up a bit :
Don't play with the hero section please.
All things are done from myside now. |
I have decreased font size by 6px because it is making website irresponsive in mobile widths. |
Mam when will this pr will get merged |
I pulled the latest changes. Now what i have to do. |
Mam please guide me |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The button is not like this, check the existing site and correct it accordingly @davyj123
Add the spacing between this
Also @davyj123 Pull and merge the latest changes from the repo, I can see you have not done anything |
Please Check it. |
@davyj123 You have not worked on the second issue. |
Done second one. |
Mam You have assigned me #114. Please tell me if i will make a github page link for that then this will expires or this will remain in working condition? |
Pull the latest changes from the repo first @davyj123 |
Alredy pulled |
Now please tell me this will get merged or any issues yet. |
Pull that again please, follow what I am saying @davyj123 Because recent changes are added to the code. |
I pulled again using fetch stream then pulled to update my local repo and then repushed my changes to my branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please resolve the merge conflicts @davyj123
Now please help me in this pull request |
This pr contain changes related to the Redesign the volunteer section #4 Redesign the volunteer section.
In this pr the volunteer section is redesigned.
Prevoius UI contain tabular representation of volunteer section which is not so good according to modern Ui's.
In this pr volunteer section is redesigned to modern flex based design with good looking tiles.
Changed To :-
Issue it resolves :- #4
Github Page Link :- https://davyj123.github.io/Blood-Buddy/