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

feat: add pronouns to account profile #44013

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Mar 5, 2024

Fix #40017

Desktop Mobile
2024-09-17_23-32 2024-09-17_23-31

image
image

  • Add free text input to enter pronouns
  • Always show the pronouns next to the displayName

@skjnldsv skjnldsv added 2. developing Work in progress feature: settings feature: profile PRs or issues related to the Profile feature (e.g. Profile page, API, etc.) labels Mar 6, 2024
Pytal

This comment was marked as resolved.

@skjnldsv skjnldsv added this to the Nextcloud 30 milestone Mar 7, 2024
@botsarenthuman

This comment was marked as off-topic.

@skjnldsv skjnldsv modified the milestones: Nextcloud 30, Nextcloud 31 Aug 8, 2024
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.

Nice, and looks good design-wise! They will also show in the visibility settings, right?

Can you show a screenshot how it will look like on the profile? I assume subline like the other things, possibly divided by the middle dot we often use? ·

@skjnldsv

This comment has been minimized.

@botsarenthuman

This comment was marked as off-topic.

@nextcloud nextcloud locked as too heated and limited conversation to collaborators Aug 8, 2024
@skjnldsv
Copy link
Member Author

skjnldsv commented Aug 8, 2024

It's just not the place to have a discussion on if a feature is controversial or not.
I'm proactively locking to not be distracted by future comments 🤟

@jancborchardt
Copy link
Member

Please make this optional, or opt-in

It's optional to put in pronouns, soo …

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the fieat/profile-pronounces branch 3 times, most recently from 8a4e7c3 to 2e04ce6 Compare September 17, 2024 22:11
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv
Copy link
Member Author

Let's go!! 🚀
I'll see how we can improve the user hover menu as a follow-up!

@skjnldsv skjnldsv requested review from susnux and Pytal and removed request for susnux and Pytal September 17, 2024 23:00
@skjnldsv skjnldsv requested review from artonge, a team, susnux and Pytal and removed request for a team September 17, 2024 23:00
@skjnldsv skjnldsv marked this pull request as ready for review September 17, 2024 23:00
@skjnldsv skjnldsv requested review from a team and removed request for provokateurin September 17, 2024 23:01
@skjnldsv skjnldsv added 3. to review Waiting for reviews feature: inclusivity 🌈 and removed 2. developing Work in progress labels Sep 17, 2024
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.

Code looks good!

No idea about LDAP part 🙈

Comment on lines +441 to +442
// Overlap avatar to top header
margin-top: -110px !important;
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// Overlap avatar to top header
margin-top: -110px !important;
// Overlap avatar to top header
margin-top: -110px !important;

@skjnldsv skjnldsv merged commit 6bcab26 into master Sep 17, 2024
177 checks passed
@skjnldsv skjnldsv deleted the fieat/profile-pronounces branch September 17, 2024 23:25
@skjnldsv
Copy link
Member Author

Code looks good!

No idea about LDAP part 🙈

I'll double check with Arthur tomorrow :)

@come-nc
Copy link
Contributor

come-nc commented Sep 18, 2024

@skjnldsv This broke the tests but the CI was wrongly green.
Please do a follow-up PR to fix the related tests.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3. to review Waiting for reviews feature: inclusivity 🌈 feature: profile PRs or issues related to the Profile feature (e.g. Profile page, API, etc.) feature: settings 🍀 2025-Spring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow setting third person pronouns in user's profile
7 participants