Skip to content
This repository was archived by the owner on Feb 8, 2026. It is now read-only.

Removing margin and padding from body#76

Closed
projectxiel wants to merge 1 commit intogofiber:mainfrom
projectxiel:pr-branch
Closed

Removing margin and padding from body#76
projectxiel wants to merge 1 commit intogofiber:mainfrom
projectxiel:pr-branch

Conversation

@projectxiel
Copy link
Copy Markdown

Without the change I've made there is a default margin on the body, or maybe its padding I didn't really check, the point is there is a gap between the outter edges of the browser and the outer edges of the UI.

I've noticed this for years, because this isn't the first time I've used this. Its been bugging me like crazy so I finally cracked and decided to fork just fix this one little thing, and really it takes a single line of js, or theoretically you could do it in CSS too.

With the change there is no longer the gap and people like me who get annoyed by little things like this can sleep easier at night.

@gaby gaby changed the title removing margin and padding from the body Removing margin and padding from the body Jan 13, 2024
@gaby gaby changed the title Removing margin and padding from the body Removing margin and padding from body Jan 13, 2024
@projectxiel projectxiel reopened this Jan 30, 2024
{{end}}

window.ui = ui
document.getElementsByTagName("body")[0].style="margin:0;padding:0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@projectxiel Why not with native style tag?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I tried that when forking, and couldn't quit get it to work, maybe someone else can

@uvulpos
Copy link
Copy Markdown
Contributor

uvulpos commented Jul 2, 2024

Solution + Workaround

Screenshot 2024-07-02 at 22 35 37 Screenshot 2024-07-02 at 22 36 05

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants