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

Remove the white space above the footer. #1362

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

aezazs-multidots
Copy link

I have removed the white space at the bottom of the footer.

Below is a screenshot.

CleanShot 2023-02-21 at 19 24 09@2x

Closes #665

@jonathanbossenger
Copy link
Collaborator

Hi @aezazs-multidots thanks for this PR

Ideally, we should be careful of removing functionality from the templates that manage pages. In this case, the PR will remove the Edit link that will appear at the bottom of the page for logged-in users.

In this case, what might be better is if you consider the styling for the entry-footer class, and look at what would happen if we adjust or remove the padding for that class.

learn-content

We would also need to make sure the change doesn't affect other pages on the site negatively.

@aezazs-multidots
Copy link
Author

Hello @jonathanbossenger

I have made the changes as you told me and committed the code.

Thanks

@courtneyr-dev
Copy link
Collaborator

Have we checked with manually editing the page in block editor before modifying theme code? Theme redesign will be shipping in the near future, so these changes will need to be reviewed again then.

@jonathanbossenger
Copy link
Collaborator

@aezazs-multidots thanks for the refresh.

I notice that the wp-content/themes/pub/wporg-learn-2020/css/style-rtl.css and wp-content/themes/pub/wporg-learn-2020/css/style.css files now have conflicts with the base branch, which I think is because they are rebuilt every time the yarn build process is run.

@adamwoodnz are you able to confirm my assumptions above? Is this something we should expect, or is there a way to prevent these conflicts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Dev] Needs Refresh Pull request needing a refresh.
Projects
Status: In Review (PRs only)
Development

Successfully merging this pull request may close these issues.

There is a lot of white space in the bottom in desktop and mobile sizes
3 participants