Xyphyn / photon

A replacement for lemmy-ui with more features, a better design, and more customizability.
https://phtn.app
GNU Affero General Public License v3.0
370 stars 42 forks source link

Design: Reduce community head stat box size #447

Open sevonj opened 3 weeks ago

sevonj commented 3 weeks ago

Is your feature request related to a problem? Please describe. So, 360px width is just a tiny bit too little to make the stats fit in a 2x2 grid. In 368px they do.

Describe the solution you'd like Reduce margin or size or something to make 2 stat blocks fit horizontally on a 360px wide screen.

Describe alternatives you've considered -

Additional context 368px 360px
image image
jansteffen commented 2 weeks ago

I think that this whole box is honestly unnescessary and even on desktop it takes up too much space: image

That's literally half the page height, even on a 1440p monitor. Just to present some info that's already being presented in the sidebar. Sure, it looks fancy with the blur effect, but in terms of navigation and useability I honestly think it's a regression.

sevonj commented 2 weeks ago

but in terms of navigation and useability I honestly think it's a regression.

I disagree, I think the the header is a better place to show the info than the sidebar, which is inaccessible on narrow screens.

Some vertical compacting on desktop too would help though.

Xyphyn commented 2 weeks ago

This info has to be accessible somewhere but I don't know where I'd put it on mobile

jansteffen commented 2 weeks ago

My suggestion would be to make in an expandable that is collapsed by default. When collapsed it only shows the banner image, community logo and name.

BrikoX commented 1 week ago

Adding community description below the name takes up valuable space for no reason and Markdown parsing from links to lists being rendered breaks the visual flow.

Full banner size, with description takes up more than half the page on 1080p desktop. It literally leaves enough space to see a single post. image

Reduced banner size, with no description still takes up 1/3 of the page on 1080p desktop. At least you can see 3 posts now. image


This info has to be accessible somewhere but I don't know where I'd put it on mobile

@Xyphyn since iOS finally added proper gesture support on PWA, you make the sidebar accessible that way. It's also already what most mobile user are used to.

Xyphyn commented 1 week ago

The gestures conflict with Android though. I'll work on the header later today, im really stressed with hwk.

sevonj commented 1 week ago

image I think closing this gap by reducing height would definitely help. It should be an easy change, except what is with mask-image affecting the other content too?

image Maybe subscribe and actions buttons could be somehow relocated to the free space in the banner which would eliminate one row. This seems like annoying to get right on all devices though, it would have to wrap below on portrait mobile.

Xyphyn commented 1 week ago

Your cropped screenshot just gave me an idea for a more compact design hold up