-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
[Request]: Move @username inline with name #159
Comments
Since this will change the layout of the status, it might be better to combine it with #62 as to avoid future revisits: additional post actions (like Edit or Delete) are in the context menu, it might be better to have them as a button since they are relatively hidden.
|
I think both the top right and bottom right in the action bar would work layout-wise, semantically I think I have a slight preference for top right. |
I'd move it up just enough so it doesn't touch the button (maybe 2px margin below the button?). Elk seems to do something similar:
My intuitition would be to try this and see how it feels in practice. Is there a branch with the current state? |
Yep: https://github.com/GeopJr/Tuba/tree/feat/status/title-redesign (it uses the flowboxes solution) |
Hmmm yeah, the spacing below the header would need to be collapsed when the username is inline, otherwise there's this weird whitespace below. Also, looking at Elk in comparison I realized that one reason why the vertical spacing looks better there is because they're reducing the height of newlines. We should probably do the same, though I'd maybe go a little bit taller than what they have. |
The header should now look as expected, but it required the status to migrate from grid to nested boxes and some margins might have been rendered useless. I think I caught everything but if you see anything that doesn't match how it looked before, let me know! One unfortunate side-effect of that change is that expanding a post is not possible anymore: before: after: If that's a deal-breaker I could try some workarounds. |
Where was expanding posts used previously? Doesn't seem like a dealbreaker to me, but maybe I'm missing something. As for moving the user name up, I'd actually go the other way and move it down a bit more, so the horizontal spacing between the avatar and the name is more similar to the vertical spacing between name and content. Alternatively we could also just increase the horizontal spacing left of the text column, maybe worth trying a few alternatives. |
Interesting, I'd never really paid attention to this, but it looks like most other apps do this expanding thing too. There's a few other differences in how other apps do it though, namely
I think it'd make sense to have something like that since everyone else does it, but then I'd also make it a bit more distinct from normal posts, with the bigger font size, different metadata layout, etc.
Agreed! |
How does this look? key notes: When expanded
|
Looks great, except for a few minor things:
|
Hmm yeah, maybe it was better with just an icon, indeed. I think the real issue is maybe that the icon looks so large next to the non-bold text. Maybe we just need smaller variants of these icons, I already noticed this in the context of the icon in the top right. Is there a list of all the icons are we using in this context, so I can draw smaller ones? Other than that this looks good to me, though I'd add middle dot separators between all the fields, not just between date and time. |
I don't think you need to re-draw them, we can decrease the pixel-size (either for both normal and expanded mode or just expanded): otherwise the icons are: (the ones with the tuba- prefix are located in |
Well, the problem is that then they're off the pixel grid and look blurry. Maybe for now try and use them at the smaller size, and we can substitute them for hinted ones later. Also, I noticed that the client name doesn't seem to be dimmed, so it looks odd when it's not a link: Other than that and the things mentioned above, this looks good to me now. |
As I said, I think with the smaller size icon-only wold be fine here. |
Opened a devkit issue to track the small icon thing https://gitlab.gnome.org/Teams/Design/icon-development-kit/-/issues/80 |
* data: small variants of icons See #159 * feat: use small icons for indicators only * feat: remove pinned regular icon it's unused * feat: use small icon for edited indicator * fix: indicator spacing * feat: conversations use open mail Co-authored-by: Jakub Steiner <[email protected]> Co-authored-by: Evangelos Paterakis <[email protected]>
Describe the request
The @username is currently below the name, which makes posts taller than they need to be, creates unnecessary whitespace on the right, and doesn't look great visually. I'd move it up next to the name, like most other clients do.
Elk:
Phanpy:
Ivory:
Implementation Details
The text was updated successfully, but these errors were encountered: