Improve the user onebox in Discourse

(Sam Saffron) #1

User onebox is a bit thin on data per:

We should improve it so it has some more data from the user card.

Embed user gallery in a post
Onebox for groups
(Maja) #2

I have been looking at this. Improving user onebox would require creating a custom template for it, correct?

(Sam Saffron) #3

Hmm it is not terrible now, but I guess there are some improvements like possibly emulating a bit of:

Posted, Joined, Read, Location all seem reasonable for the card.

You would need a custom template, yes.

@codinghorror what do you think here?

(Jeff Atwood) #4

Oh I see yes sure the user onebox is pretty weak. Totally fine go for it.

(Chris Beach) #5

Also doesn’t seem to work for all of my users. Example:

… but does work for me

Is this another problem where there’s no description and Onebox fails to render anything?

(Jeff Atwood) #6

That would be my guess, yes.

(Sam Saffron) #7

In theory we could have the onebox description here be synthesized from the user activity or something. "member since: 2017-1, 22 posts, last posted yesterday, avid reader"

The biggest omission there is that it’s not prioritizing “long name” which I think is correct for public shares. It should be “Chris Beach” in the public onebox, weird that the username is there.

(Maja) #8

This is how it looks like with my changes. Thoughts?
I wonder if it’s really ok to include “posted” and “read recently” since it’s going to be out of date pretty soon after it’s baked?

(Jeff Atwood) #9

Looks good!

Yeah, great point, we try to avoid highly time-specific data in oneboxes as they are typically baked only at the time of posting. Perhaps some other stats would be less time dependent here? Join date seems safe enough at least.

(Maja) #10

I have added a PR add a custom user onebox by majakomel · Pull Request #5542 · discourse/discourse · GitHub, slightly different than the screenshot above, without the time specific stats. Other stats that could be relevant (like created posts/topic count) are also changing fast, so I haven’t included them.

(Sam Saffron) #11

This looks excellent to me! Thanks @maja