Add user additional info field for share autocomplete#29334
Conversation
|
@pmaier1 please try it out. We might want to tweak the visual design a bit. |
jvillafanez
left a comment
There was a problem hiding this comment.
Code looks good, not tested
|
We should also test with mobile devices (or small window size). UX might change |
|
@felixheidecke any suggestion the visual design ? Now having the email address appearing there makes the sidebar look more loaded. Eventually we might want to move those permissions to a sub-panel/accordion but out of scope for this PR/feature. |
|
To test please go to the settings page under the "Sharing" section where you can select what information to display with the sharee in autocomplete and in the rows. |
lordelix
left a comment
There was a problem hiding this comment.
I added a little CSS. Looks good from a UI pov. 👍
|
seems I need to look into test failures |
|
Added PHP unit tests, squashed, rebased. Quick rereview appreciated @jvillafanez |
a6feeeb to
61953ed
Compare
Codecov Report
@@ Coverage Diff @@
## master #29334 +/- ##
============================================
+ Coverage 60.45% 60.46% +0.01%
- Complexity 17222 17231 +9
============================================
Files 1032 1032
Lines 57360 57381 +21
============================================
+ Hits 34677 34698 +21
Misses 22683 22683
Continue to review full report at Codecov.
|
|
stable10: #29457 |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
Related Issue
Fixes #28979
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: