Skip to content

GH-87 fix: header gap#86

Closed
Dobya11 wants to merge 2 commits intoHytaleModding:mainfrom
Dobya11:main
Closed

GH-87 fix: header gap#86
Dobya11 wants to merge 2 commits intoHytaleModding:mainfrom
Dobya11:main

Conversation

@Dobya11
Copy link
Copy Markdown
Contributor

@Dobya11 Dobya11 commented Mar 17, 2026

Added to app.css
gh-87

@Dobya11 Dobya11 changed the title fix: header gap GH-83 fix: header gap Mar 17, 2026
@hytalemodding hytalemodding bot mentioned this pull request Mar 17, 2026
@ItsNeil17 ItsNeil17 changed the title GH-83 fix: header gap GH-87 fix: header gap Mar 17, 2026
@ItsNeil17
Copy link
Copy Markdown
Member

!bot sync

@hytalemodding
Copy link
Copy Markdown

hytalemodding bot commented Mar 17, 2026

Synced with issue #87

📋 Labels: No labels on linked issue
🎯 Milestone: No milestone on linked issue

Copy link
Copy Markdown
Member

@ItsNeil17 ItsNeil17 left a comment

Choose a reason for hiding this comment

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

Could you please edit the tailwind classes instead of adding to the CSS file?

Public Index Page (/mod/{name}): https://github.com/HytaleModding/wiki/blob/main/resources/js/pages/Public/Mod.tsx
Mod Docs Page (/mod/{name}/{page}): https://github.com/HytaleModding/wiki/blob/main/resources/js/pages/Public/Page.tsx

Copy link
Copy Markdown
Member

@okunamayanad okunamayanad left a comment

Choose a reason for hiding this comment

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

I'm unsure if this is a "works on my machine" problem but this looks totally broken on me:

Before:
Image

After:
Image

Although I have to mention, we are going to redesign this entire page as it was sped up work meant to be replaced (there are a couple of these pages around here, they're for @ItsNeil17 to be able to continue working on backend without being bottlenecked by frontend team's speed).

@ItsNeil17
Copy link
Copy Markdown
Member

Closing in favor of #97

@ItsNeil17 ItsNeil17 closed this Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants