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

Improve spacing on mobile landscape mode #3

Closed
aredshift opened this issue Oct 26, 2023 · 8 comments · Fixed by #9
Closed

Improve spacing on mobile landscape mode #3

aredshift opened this issue Oct 26, 2023 · 8 comments · Fixed by #9
Assignees
Labels
good first issue Good for newcomers

Comments

@aredshift
Copy link
Contributor

When mobile is flipped to landscape mode the links-bar looks kind of crowded. I think we could improve the proportions (possibly decrease the margin between the header and the quote)

image

@aredshift aredshift added the good first issue Good for newcomers label Oct 26, 2023
@mrbjnb
Copy link
Contributor

mrbjnb commented Oct 31, 2023

I want to try out but I am not sure about the requirement you mentioned. Is it adjusting the margin so the header and the quotes spacing far away from each other?

@aredshift
Copy link
Contributor Author

Hi @mrbjnb, thanks for offering to contribute! I apologize for the lack of clarity - we're a small group of friends getting started on a few fun projects together and we just put together this website to be a hub for our group projects, so we haven't formalized an issue or PR template yet.

You're most welcome to take on this issue if you'd like to help!

To clarify the issue: the pink-colored links bar at the bottom of the page is right up against the bottom of the screen, and I think it would look better if there was more empty space on the bottom. I think this empty space should come from between the header and the quote.

One thing to keep in mind is that we're trying to keep the website design responsive, so while making this change, please be sure to test it on different sizes of screen. If you're new to these concepts and want some more direction I'd be happy to give you a few pointers. Otherwise I'll assign this issue to you and leave you to it!

@mrbjnb
Copy link
Contributor

mrbjnb commented Oct 31, 2023

Oh I see, thank you for the assignment, I will try to figure it out. 🫡

@mrbjnb
Copy link
Contributor

mrbjnb commented Nov 3, 2023

image
Is this good enough? @aredshift

@mrbjnb
Copy link
Contributor

mrbjnb commented Nov 3, 2023

Also I should commit it to the dev branch right? I am new to GitHub development.

@mrbjnb
Copy link
Contributor

mrbjnb commented Nov 3, 2023

I have no idea how to create a pull request, can you help me?

@aredshift
Copy link
Contributor Author

Hey @mrbjnb! Great start, the links-bar looks like it has more breathing room at the bottom of the page now. One note: I'd like to see the header & subheader look bigger on the screen and the quote look smaller.

As for submitting a pull request, here is some helpful documentation for best-practices for contributing to open source projects on Github: https://docs.github.com/en/get-started/quickstart/contributing-to-projects.

Basically what you'd normally do is:

  1. Create a fork of our project under your ownership (eg. mrbjnb/website, but you can rename it something different if you'd like)
  2. Clone your fork
  3. Create a branch for your work from the forked repo, for example 3-improve-landscape-spacing (but you can name it what you like)
  4. Push your changes to your fork
  5. Submit a PR from your fork's feature branch to our project's main branch

Then we can review/update and then merge it when everything looks good!

One convoluting factor is that I'm guessing you've cloned our repo to make and test the change. Since this project is pretty small and I imagine you don't have a complex git commit history, you should simply be able to copy and paste the contents of the files you changed and then re-commit them to your fork's feature branch. If however you want to preserve your commit history or just want to learn how to move a branch between repositories, lmk and I will give you some instructions.

In the future, we'll be sure to make a contributing instructions file for our repo so this is all much more straightforward. :)

Thanks again for helping out and asking questions!

@mrbjnb
Copy link
Contributor

mrbjnb commented Nov 3, 2023

Oh I see, thank you for your details answered. I will improve it and try to make a pull request soon.

mrbjnb added a commit to mrbjnb/website that referenced this issue Nov 3, 2023
@aredshift aredshift changed the title improve spacing on mobile landscape mode Improve spacing on mobile landscape mode Nov 4, 2023
@aredshift aredshift linked a pull request Nov 4, 2023 that will close this issue
aredshift pushed a commit that referenced this issue Nov 9, 2023
* improve spacing on mobile landscape mode #3

* Make some changes in landscape and add portrait mode

Added portrait mode, adjusted the size in landscape mode and added quotes back (reset.css remove it)

* Clarify modes, add S size and modify reset.css

* Fix tablet size mode, reset margin for quote
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants