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

Landing page #32

Merged
merged 12 commits into from
Nov 12, 2023
Merged

Landing page #32

merged 12 commits into from
Nov 12, 2023

Conversation

Kirk-Esterline
Copy link
Contributor

Commit coauthored with Ahmed Hamdy

Description

Built out the landing page to resemble the design layout.
We added a ul, two imgs, as well as necessary tailwind classes to page.jsx.

Testing

We tested on different browsers and screen sizes to check responsiveness.

Did you change the schema?

  • Yes
  • [ X ] No

Issue

Closes #20

Kirk-Esterline and others added 10 commits November 2, 2023 19:57
…to the homepage to reflect the layout. Styling has only been started. Padding and margins need to be adjusted. The included screen shot was only a test and needs to be replaced with real screenshots and placed on the layout properly. No changes have been made to the 'header' at this point.
…tion to get the ul and img to be side by side. Adjusted syles to better fit the design. Added the 100Freelancers logo to public but it is not being used anywhere yet.
… page.jsx. Tailwind code is still divided between the global.css and page.jsx. Next steps are to continue the styling on the screenshots to match the example layout.
…nline on page.jsx. Finalized the visual adjustments to the two screenshots to have them reflex the design layout. Ahmend restyled the Sign In button as well as repositioned the logo and button within the header.
Copy link

vercel bot commented Nov 10, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @devlarabar on Vercel.

@devlarabar first needs to authorize it.

Copy link
Owner

@devlarabar devlarabar left a comment

Choose a reason for hiding this comment

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

Hey Kirk and @AhmedHHamdy ! I really appreciate your hard work on this! The desktop layout looks beautiful. Very clean and crisp.

Something that stands out to me is the responsiveness. The layout doesn't adjust for mobile as in Karissa's original design referenced in the landing page issue. This is what it looks like when I resize:

Screenshot 2023-11-10 141756
Screenshot 2023-11-10 141806
Screenshot 2023-11-10 141829

It starts to look off between 860-1100px wide. I know Karissa's mobile design includes swiping between mobile screenshots, but for simplicity, it would be fine for now to just include one screenshot and we can enhance it in a future sprint.

For the "mid" sizes (smaller monitors and tablets), we can probably include the desktop and mobile screenshot images, but move them from the right side of the page to below the "View", "Add", and "Track" text. Then for super small screens like mobile, omit the desktop screenshot entirely and only show the mobile screenshot. Hope that makes sense - let me know if I should clarify!

Aside from this, I noticed some comments in globals.css that can be removed to keep the file clean and readable.

Thanks again for working so hard on this, and please feel free to ping me in Discord any time if you need help!

…k points in page.jsx to add responsiveness to the landing-page.
@Kirk-Esterline
Copy link
Contributor Author

Thank you for the quick and clear feedback!

I added several break points to make the page responsive to screen size. On smaller screen sizes the images are beneath the "view" "add" and "track" text. On larger screens the text and images are side by side however, the images will respond by either being stacked or overlapped depending on screen size. I made that decision to make the layout feel less cramped in the 'large' breakpoint without needing to make either image smaller.

Let me know if additional changes are required.

…e longer without causing too much overlapping. And adjustd the styling on mobile devices to be a bit more mobile appropriate.
@Kirk-Esterline
Copy link
Contributor Author

With Ahmed's help we added aditional break points to keep the text and images side by side on a wider range of screens, and adjusted the styling on mobile screen sizes to be more mobile specific.

Copy link
Owner

@devlarabar devlarabar left a comment

Choose a reason for hiding this comment

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

Excellent! Thank you both so much for implementing the changes quickly. Everything looks great to me 👍

@devlarabar devlarabar merged commit c056a92 into devlarabar:main Nov 12, 2023
1 check passed
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.

Landing Page Design & Construction
2 participants