Skip to content
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

Completely hide "Users Online" section from Info Center when show_who… #5641

Closed
wants to merge 3 commits into from

Conversation

m4z
Copy link

@m4z m4z commented Apr 29, 2019

… is disabled

Fixes #5632 (does what the help text seems to imply)

Christopher 'm4z' Holm added 2 commits April 29, 2019 21:13
@SychO9
Copy link
Contributor

SychO9 commented Apr 29, 2019

that setting is only for the online list, if we go down the road of hiding online status then we'd also have to apply this everywhere including the online status on topics and profiles,

So I think the help text needs to be edited to reflect what the setting actually does, not the other way around

@m4z
Copy link
Author

m4z commented Apr 29, 2019

Individual users can choose to turn off their Online status (if the admin allows it), or are you talking about more?
I'm a big fan of disabling such... let's call it "statistics" instead of "user tracking features". 😉 Especially for closed forums, I don't want to leak such details to "externals".

(But if this isn't in line with the vision of SMF, I'll try putting this into a mod instead and will include a more concise text for this issue.)

@m4z
Copy link
Author

m4z commented Apr 29, 2019

PS: It would have been nice if you (or anybody) would have said which option they prefer in the issue before I spent all evening with PHP. 😉 The text option would've been "a few hours easier".

@SychO9
Copy link
Contributor

SychO9 commented Apr 29, 2019

Well there are 101 issues you know, we can't tackle them all at once, we try to figure them out by order of their priority (milestones) and it isn't easy when the people contributing don't have much free time.

in any case, what I said is only an opinion, please don't take it as a "this is what has to be done" 😅

PS: It would have been nice if you (or anybody) would have said which option they prefer in the issue before I spent all evening with PHP. 😉 The text option would've been "a few hours easier".

We really do appreciate the help, the more people the better, but please don't be rude about it, I'm sorry you spent a lot of your time on it, but that's what we all do here as well.

@m4z
Copy link
Author

m4z commented Apr 29, 2019

I understand that and didn't mean to be rude at all (it was a learning/refresher experience anyway and I chose to do it, so no harm done); I just mean any dev could have voiced an opinion (earlier) in the issue (before I began investing effort in one of the solutions). Trying to help, but when there's no comment at all, I'm just flying blind and hitting more things than necessary. 🚀

@sbulen sbulen added the Theme label May 31, 2019
@sbulen sbulen added this to the Final milestone Jun 6, 2019
m4z pushed a commit to m4z/SMF that referenced this pull request Oct 31, 2019
@m4z
Copy link
Author

m4z commented Oct 31, 2019

This is superseded by #5852, but either I don't know the GitHub magic word for that or such a feature doesn't exist. 😉

I'd like to keep this around though, and work it into a full-fledged "Disable the Info Center" feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Misnomer $modSettings['who_enabled'] and incorrect $helptxt['who_enabled']
3 participants