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.

Sign up for GitHub

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

Jump to bottom

About Us Page Mobile Layout #50

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

About Us Page Mobile Layout #50

wants to merge 3 commits into from
+63 −0

Conversation

aratakon
Copy link
Collaborator

No description provided.

@aratakon aratakon linked an issue Mar 11, 2025 that may be closed by this pull request
About Us Mobile #33
Open
@naomitzhao naomitzhao self-requested a review March 17, 2025 23:07
naomitzhao
naomitzhao reviewed Mar 18, 2025
View reviewed changes
Copy link
Collaborator

@naomitzhao naomitzhao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Clinic Roles and Annual Health Fair sections are perfect-- well done. I notice there's still some weird behavior for the About Us cards-- did you need help with that?

@aratakon
Copy link
Collaborator Author

The Clinic Roles and Annual Health Fair sections are perfect-- well done. I notice there's still some weird behavior for the About Us cards-- did you need help with that?

For the About Us cards I wasn't sure on how to move the image on top of text, I would like help with that if possible. I was able to move them from left to the center but wasn't sure on how to move them on top of the text.

@naomitzhao
Copy link
Collaborator

The Clinic Roles and Annual Health Fair sections are perfect-- well done. I notice there's still some weird behavior for the About Us cards-- did you need help with that?

For the About Us cards I wasn't sure on how to move the image on top of text, I would like help with that if possible. I was able to move them from left to the center but wasn't sure on how to move them on top of the text.

I responded to you on Slack. Let me know your thoughts and I can either take over this issue for you or just provide you with assistance :)

@C3viche C3viche changed the title 33-about-us-mobile About Us Page Mobile Layout Mar 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@naomitzhao naomitzhao naomitzhao left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

About Us Mobile
2 participants
@aratakon @naomitzhao