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

Better wordings in the new user modal #44931

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

Jerome-Herbinet
Copy link
Member

I've made some wording improvements ; those are more clear, IMO.
Should do what you see below. Tests from reviewers required ; thanks by advance to them.

Before

2024-04-19_10-51

After

2024-04-19_10-52

Checklist

Copy link
Member

@Pytal Pytal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea though seems a bit long, will have to see what designers think :)

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added 2 suggestions, simplifying the language.

Also, we shouldn’t ever use the "(s)" singular/plural writing like "group(s)". Either we write only the plural (as in my suggestion), or we programmatically check:

  • If nothing or multiple are put in, use plural
  • If only 1 is put in, use singular

apps/settings/src/components/Users/NewUserModal.vue Outdated Show resolved Hide resolved
apps/settings/src/components/Users/NewUserModal.vue Outdated Show resolved Hide resolved
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me! :)

@jancborchardt jancborchardt requested review from susnux and Pytal June 13, 2024 14:59
@jancborchardt
Copy link
Member

@Pytal @susnux could you check again? :)

@solracsf solracsf added this to the Nextcloud 30 milestone Jun 18, 2024
@Pytal
Copy link
Member

Pytal commented Jun 19, 2024

Looks good! Can you squash the commits @Jerome-Herbinet?

Jerome-Herbinet and others added 2 commits June 25, 2024 00:10
Co-authored-by: Jan C. Borchardt <925062+jancborchardt@users.noreply.github.com>
Co-authored-by: Pytal <24800714+Pytal@users.noreply.github.com>
Co-authored-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the Jerome-Herbinet-better-wordings-in-new-user-modal branch from b27fdb6 to 1c168da Compare June 24, 2024 22:13
@susnux
Copy link
Contributor

susnux commented Jun 24, 2024

Squashed and rebased

@susnux susnux requested a review from Pytal June 24, 2024 22:13
Copy link
Member

@Pytal Pytal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
💃💃

@susnux susnux merged commit 56462e9 into master Jun 24, 2024
110 checks passed
@susnux susnux deleted the Jerome-Herbinet-better-wordings-in-new-user-modal branch June 24, 2024 22:52
@blizzz blizzz mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews design Design, UI, UX, etc. feature: users and groups
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants