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

style: making some rows look nicer #1099

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

liwoyadan
Copy link

I know this was mentioned a few times in the server haha so I've gone ahead and did a quick few style edits...these edits just add no-gutters to a handful of rows to eliminate some of the preset horizontal padding on child columns and adjusts a few of the column numbers. The following pages received changes:

User Profile

Character Information Page (for mobile, tested at 320px)

This, of course, doesn't account for site owners who may use alternative fonts or change the size of headings themselves in the CSS...but in that case you'd want to do further edits yourself regardless, hahaha.

- User profile rows no longer are so small that they break heading text on >= 992px viewports (as their biggest breakpoint is lg)
- Character information pages (image info and stats tab) have also had rows adjusted to prevent breaking headings
Copy link
Collaborator

@itinerare itinerare left a comment

Choose a reason for hiding this comment

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

Love this, thank you!

@itinerare itinerare added the reviewed Pull requests that have received community review and are pending merge label Oct 29, 2024
@itinerare itinerare merged commit 79397de into corowne:release/v3.0.0 Oct 29, 2024
2 checks passed
SpeedyD added a commit to SpeedyD/lorekeeper that referenced this pull request Oct 30, 2024
…ase-to-dev

* corowne/release/v3.0.0:
  fix(galleries): make some queries more efficient (corowne#1098)
  fix: improve user profile, character page row styling on smaller viewports/mobile (corowne#1099)
@liwoyadan liwoyadan deleted the fixes/row-adjustments branch November 24, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewed Pull requests that have received community review and are pending merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants