layer5io / layer5

Layer5, expect more from your infrastructure
https://layer5.io
Apache License 2.0
840 stars 1.12k forks source link

Improve Mobile-Friendliness of "Learn Cloud Native" Section #5645

Open Ashparshp opened 1 week ago

Ashparshp commented 1 week ago

Description

Expected Behavior

Steps to Reproduce:

Screenshots

Screenshot_2024-07-07-16-48-05-181_com android chrome

Environment:


Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Join the Layer5 Community by submitting your community member form.

MUFFANUJ commented 1 week ago

Please assign this to me

vishalvivekm commented 1 week ago

@MUFFANUJ I notice two your PRs submitted without the Issues being assigned to you. Please understand that it's important for us to uphold community guidelines.

I understand that you may not have had the chance to go through the contributing guides and code of conduct outlined in the community handbook. Please go through these documents and I hope you'll keep them in mind for future.

vishalvivekm commented 1 week ago

As for this issue, @Ashparshp Let's make call for volunteer on Websites call tomorrow. Adding this into meeting minutes.

r-kundan commented 1 week ago

Please assign this to me

MUFFANUJ commented 1 week ago

@MUFFANUJ I notice two your PRs submitted without the Issues being assigned to you. Please understand that it's important for us to uphold community guidelines.

I understand that you may not have had the chance to go through the contributing guides and code of conduct outlined in the community handbook. Please go through these documents and I hope you'll keep them in mind for future.

I totally understand my mistake in the prior Pull requests and want to apologize for that, but this one wasn't assigned to anyone so can't I take this one?

jaysomani commented 1 week ago

Can i work on this?

iamclintgeorge commented 1 week ago

Can you assign me this issue?

Ashparshp commented 1 week ago

@iamclintgeorge Also, for bigger screens as well the background is too much we can reduce it as well!

iamclintgeorge commented 1 week ago

@Ashparshp Sure, i will look into it!!

iamclintgeorge commented 1 week ago

Screenshot from 2024-07-09 21-04-40

@Ashparshp Is this fine?

Also as per your request, i decreased the height of the background from 35rem to 30rem.

Ashparshp commented 1 week ago

Thanks, @Harsh-svg988! Please go ahead and generate a pull request so we can review it better. By the way, are you also increasing the font size?

iamclintgeorge commented 1 week ago

Yes i had increased it. Thanks for pointing out, i will the changes to make it look consistent with the other headings.

Ashparshp commented 1 week ago

Thanks for pointing that out. From where are you maintaining the consistency with the other headings?

iamclintgeorge commented 1 week ago

I had set the size of the "Learn Cloud Native" heading relatively more than the other headings that are present in the same page such as the "Service Mesh Books" Heading. I have made the required corrections.

Screenshot from 2024-07-09 21-43-18

Screenshot from 2024-07-09 21-44-10