-
Notifications
You must be signed in to change notification settings - Fork 166
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
More appropriate circle section wording #3441
base: main
Are you sure you want to change the base?
More appropriate circle section wording #3441
Conversation
"user" groups instead of "contact" groups Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #3441 +/- ##
=====================================
Coverage 2.10% 2.10%
=====================================
Files 88 88
Lines 5283 5283
Branches 1464 1464
=====================================
Hits 111 111
Misses 5057 5057
Partials 115 115 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed it does not take "Contact groups" here but just the regular groups from the accounts. It should be simply called "Groups", as we call it like that elsewhere too.
It also needs to be adjusted in the "Add members" modal for the subheading, if you'd like to do that as well @Jerome-Herbinet :)
@jancborchardt I understand what you're saying and the arguments behind it, and I'll let you have the last word on what needs to be done. :-) Nevertheless, I'm giving you my impressions as a user, before making any last changes. If, in the same interface, you have "Contact groups" in the left-hand column (linked to the contacts in the address books), and "Groups" in the middle column to make up the Circles, there's bound to be confusion, because if in one case you specify what type of group it is, but not in the other, all doubts are allowed and users will legitimately think that these are also contact groups. In my opinion, the initial naming rules should be departed from here, because in the same interface there are two types of group: contact groups and user groups. Elsewhere in Nextcloud, when there can be no confusion, we can make do with "Groups" without any problem, but not in this specific case, which I think is an exception. In French, we have an expression to describe that : "This exception is the one that proves the rule." :-D What do you think ? |
You're right @Jerome-Herbinet. Actually I'm a bit confused since I thought Circles allows you to add both "account groups" and "contact groups" to a circle. The omission of "contact groups" is a bit strange since contacts individually can be added. @ArtificialOwl any specific reason here? |
It has never been officially requested :-) |
Hello @jancborchardt @ArtificialOwl, |
@jancborchardt @ArtificialOwl , any news ? |
"user" groups instead of "contact" groups because here are only listed user groups and not contact groups (if I'm not mistaking).