Skip to content

fix(footer): changes to Layout.js and _document.js #43

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

Merged
merged 3 commits into from
Mar 29, 2023

Conversation

darkterminal
Copy link
Member

πŸ““ What Your Metaphor

Changes to Layout.js and _document.js

πŸ‘¨β€πŸ’» Suggest for Community (write if any)

  • Changed Layout.js to remove unnecessary class name and improve readability
  • Added padding to the bottom of the body in _document.js for better UI

βœ”οΈ Check List (Check all the applicable boxes)

  • My metaphor is not written before from anyone
  • My metaphor have follow writing rules of the community.
  • The title of my pull request is a short description of the requested merge.

πŸ“„ Note to reviewers (write if any)

πŸ“· What Your Punk Signature?

Mobile Preview

Desktop Preview

Add fixed position to the footer, so it always stays at the bottom of the page. This improves the layout of the page and makes it more user-friendly.
Remove the 'flex-1' class from the main element in the Layout component as it is not needed and can be simplified.
Previously, the body element in _document.js did not have any padding applied to it, causing some content to be obscured by the footer. This commit adds a relative position to the body element, as well as a bottom padding of 20px, to ensure that all content is visible.
@vercel
Copy link

vercel bot commented Mar 29, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated
metaphore βœ… Ready (Inspect) Visit Preview πŸ’¬ Add your feedback Mar 29, 2023 at 4:55PM (UTC)

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello Punk! It's great having you contribute to this project

Welcome to the community :neckbeard:

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our GitHub Organisation - we help and encourage each other to contribute to open source little and often :neckbeard:. Any questions let us know.

@darkterminal darkterminal requested a review from mkubdev March 29, 2023 16:55
@darkterminal darkterminal changed the title Changes to Layout.js and _document.js fix(footer): changes to Layout.js and _document.js Mar 29, 2023
@darkterminal darkterminal merged commit 9bf6e47 into StreetCommunityProgrammer:main Mar 29, 2023
github-actions bot pushed a commit that referenced this pull request Mar 29, 2023
# [1.2.0](v1.1.0...v1.2.0) (2023-03-29)

### Bug Fixes

* **footer:** changes to Layout.js and _document.js ([#43](#43)) ([9bf6e47](9bf6e47))
* navbar link for mobile now match desktop links ([3475205](3475205))
* remove  class\ footer need to be refactored ([d94b718](d94b718))

### Features

* add [@darkterminal](https://github.com/darkterminal) suggestion ([cf9c2b5](cf9c2b5))
* Add Tailwind responsive utility class ([#37](#37)) ([0fa0a38](0fa0a38))
* adding golang brand icon ([6f6a58a](6f6a58a))
* adding golang quotes ([dbf514b](dbf514b))
* display no story page ([c35a029](c35a029))
* **footer:** add fixed position to the footer ([6267e3b](6267e3b))
* **resp:** fix homepage responsivness ([00404bd](00404bd))
* **responsive:** add responsivness to pages ([96c7e0a](96c7e0a))
* update _document.js to add padding to body ([11195cc](11195cc))
@github-actions
Copy link
Contributor

πŸŽ‰ This PR is included in version 1.2.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant