Closed CTalvio closed 10 months ago
Sounds like for now this is on hold, but copying over my comments from Matrix/ #431:
I know ours won't look exactly like in Bean (defer to what feels right for Thunder), but pasting them as an example here. Thanks for taking an exploratory stab at this!
Great start! Agree with @machinaeZER0's comment.
Progress. I'm moving the saved/pinned icons to the title, they make sense there, and the lower part of the cards is getting cramped.
Nice progress! I don't mind saved/pinned in the title, but maybe after would be better? I think (a) this would match other UIs, (b) there wouldn't be a problem of overflow since we wrap, and (c) it would keep the beginning of the title in a consistent place (no UI clutter to skip over before reading the title). Also I wonder if the pin icon could be a tiny bit smaller. 😊
Span doesn't handle the icon well at the end, it has to be made super tiny to fit.
Maybe the whole title could be tinted green for pinned posts, like the webUI?
If you don't mind me weighing in - I do like the idea of putting the pin at the end of the title, but if that's not possible I wouldn't worry about it. Tinting the text green would be my vote, since (to your point) it matches the webui and doesn't feel too "loud" (though if the pin icon stays as well I would move it next to the title - looks like right now the Saved star places itself in between?)
Alternatively, would it be worth trying something with chips in this situation? If a pinned title becomes a green chip/badge, would that do anything to solve the issue where the pin icon has to be made super small to fit? Probably way more work unless something can be carried over from the work on username display in comments (and it might be unnecessarily loud)
I kinda like it when the pin/star is small and at the end. ALSO BADGES!
Oh wow!
I think the link one looks perfect - I might shrink the other icons a little within their respective badges, personally. Do they all have slightly different badge colors, or just the link one?
Thank you for working on that!
I kinda like it when the pin/star is small and at the end. ALSO BADGES!
This looks absolutely amazing! 🎉
Allright, this is ready to go! Let me know @hjiangsu and @micahmo if you find anything that needs fixing.
Looks good! The only thing is to address the merge conflicts here (I think it most likely has to do with changing the textScaleFactor to use the correct one)
E.g., use metadataFontSizeScale
instead of contentFontSizeScale
for the post card metadata text!
@CTalvio Could you double check the formatting - it didn't pass the checks!
431
511
Lots of small changes to both post view types