-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Conversation
bd95aa8
to
3d77190
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
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.
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? ·
This comment has been minimized.
This comment has been minimized.
This comment was marked as off-topic.
This comment was marked as off-topic.
It's just not the place to have a discussion on if a feature is controversial or not. |
It's optional to put in pronouns, soo … |
d934fba
to
64549d0
Compare
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
8a4e7c3
to
2e04ce6
Compare
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
2e04ce6
to
b24e02e
Compare
Let's go!! 🚀 |
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.
Code looks good!
No idea about LDAP part 🙈
// Overlap avatar to top header | ||
margin-top: -110px !important; |
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.
// Overlap avatar to top header | |
margin-top: -110px !important; | |
// Overlap avatar to top header | |
margin-top: -110px !important; |
I'll double check with Arthur tomorrow :) |
@skjnldsv This broke the tests but the CI was wrongly green. |
Fix #40017