Skip to content

Containerized asset grid#308

Open
PhilippKosarev wants to merge 5 commits intoAchetaGames:mainfrom
PhilippKosarev:ui-improvements
Open

Containerized asset grid#308
PhilippKosarev wants to merge 5 commits intoAchetaGames:mainfrom
PhilippKosarev:ui-improvements

Conversation

@PhilippKosarev
Copy link
Contributor

This is a bit experimental, so I'm curious what you think of this.

Before

ye old

After

ye new 12
image

@aknarts
Copy link
Contributor

aknarts commented May 16, 2025

So while I like this and I think this was the initial look, the issue is that some asset names are stupidly long and that breaks the UI in a big way.. that is why there was the ellipsis on the name.. otherwise the margin adjustment would be fine.

@PhilippKosarev
Copy link
Contributor Author

Unfortunate, but makes sense. I've tried changing the GtkLabels lines property, but then when the label is wrapped - its requested height doesn't change, so it slides onto other elements in the ui :(

Anyhow, I've added back the ellipsis (along with some other minor styling tweaks).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants