Dear David, i find a problem
when I select a group, scrolling the mouse to load more, all users will be loaded directly instead of selected group members.
can you have a look on this
Dear David, i find a problem
when I select a group, scrolling the mouse to load more, all users will be loaded directly instead of selected group members.
can you have a look on this
Thank you for another great component
There is a small glitch with the new Dark Mode setting:
Dark Theme:
Light Theme with automatic Dark Mode colour scheme enabled:
And a request: is it possible to add two settings to this component for
I’d like to display users who were the most active readers in the past year at the top by setting default timespan to ‘Year’ and sort descending by ‘Read’.
Is this something that can be easily added?
Hi @david This theme component continues to add value to our Discourse, but recently a known bug/feature caused some really confusion and pain for us.
The known bug/feature: The logged in user is always returned in the results even if they don’t match the filter condition.
In the case below, my user isn’t part of the ruby group, but I show up.
What’s possible in terms of changing the theme—so that the user viewing the directory, doesn’t see themselves in results where they don’t belong?
Thanks so much for digging into what we could do to help change this behavior. We’re on Discourse-based hosting.
Hi @alxpck, sorry for the delayed response here!
This is actually coming from the core ‘user directory’ behaviour - you can see it if you open /u here on Meta and try filtering by username. The current user will always be shown in the list, regardless of what you filter by. For example
But you’re right that it becomes super confusing when filtering by groups. If someone is deliberately filtering like that, they probably don’t want to see themselves (unless they match the filter).
I opened a PR in core here for discussion: UX: Do not include current user in filtered directory results by davidtaylorhq · Pull Request #11310 · discourse/discourse · GitHub
Let’s see how that goes - if we decide not to make that change in core, we can look at a workaround for this theme component.
Thank you so much @david. I really appreciate you looking at this from both a core Discourse and a User Card Directory theme perspective.
Either way you come at it, we know making a change would be immensely valuable to our community.
No worries about the delay—2020 am I right?
I merged @david’s change… your sites will get it over the next week or so.
The only open item is deciding on fidelity of “always show me” if I visit the directory and am not filtering anything.
Are you still interested in the “don’t every show myself first when I visit the /u
page (even without a filter)” feature, we could sort of work around it in the component.
Wonderful! Thank you @sam @david and team!
Re: “always show me”
/u
page layout which is a kind of stats-driven leaderboard. That way it’s easy for them to measure themselves against the other people on the Discourse./my/summary
or click on their photo.I can’t speak to all the other use cases for other people’s Discourses, but for our needs we’d definitely prefer if it was possible to avoid showing people their own profile (with or without a filter).
I can’t speak for others and have not heard much complaining about the user directory, but personally I also do not use it as a leaderboard and like it mostly for discovery.
Maybe a toggle could be provided to make it go away?
Possibly, we need a call fro @codinghorror if he wants site setting in core of if this is a theme component thing, it has popped up 2-3 times over the years
At a minimum we can adjust core in such a way that a theme component can amend this
I am not sure, the “grid of cards” thing seems rather site specific. I think it is fine for it to be a site setting? I don’t have strong feelings about this, but I don’t think it makes sense as a default.
Here’s a PR which adds a ‘hide current user’ setting to the theme component. If enabled, it will always hide the current user from view
Does it handle the case where you actually want to see current user? eg you searched for yourself… you are at the top of the leaderboard … etc?
Probably good enough as is for @alxpck … but thinking out loud.
It unconditionally hides the current user, and it’s a default-disabled theme setting. I think that’s acceptable for the user-card layout? But definitely wouldn’t work for a leaderboard.
Thank you!
This is a wonderful improvement.
We would prefer the granularity that @sam mentioned (cases where you actually want to see the current user), but removing people from searches where they don’t meet the filter criteria was the problem that brought us here—so I really appreciate the theme setting to hide the current user.
I’m seeing at least a few of my users’ cards appearing more than once… eg “andy1” and “blane” among others on this page:
update: Something funny is going on. I saw it in cognito on Chrome, but I don’t see the dupes logged in when using Safari. This is going to be hard to track down…
Hi Craig - thanks for the report. Looks like this is a problem with Discourse core - it happens even with the user cards component disabled.
The root of the problem is that all your users currently have ‘0’ values for the metrics, and our sorting was inconsistent. I’ve opened a PR to fix the issue, and will get it deployed to your site once it’s merged. Technical details are in the PR if you’re interested
That’s fixed, and I’ve deployed the core change to your site @craigconstantine. Let us know if you’re still seeing any issues.
That’s fixed, sorry for the long wait! Update the theme component in your admin dashboard and things should look good on light/dark themes