User list : User name and Display name visibility improvement

[Jerome-Herbinet]

Signed-off-by: Jérôme Herbinet [email protected]

Signed-off-by: Jérôme Herbinet [email protected]

[Jerome-Herbinet]

@szaimen (following our conversation a few minutes ago) I've closed my old messy PR (#33669) and I've created this brand new one, with exactly the same changes. It of course takes in account the design feedback of @jancborchardt. I think that everything should be OK (hoping it's not too late for Nextcloud 25).

[szaimen]

@szaimen (following our conversation a few minutes ago) I've closed my old messy PR (#33669) and I've created this brand new one, with exactly the same changes. It of course takes in account the design feedback of @jancborchardt. I think that everything should be OK (hoping it's not too late for Nextcloud 25).

Thank you! Yes, the PR looks good to me. However I fear we need the compiled js. Can you please run npm run build and commit the changes to this branch?

[Jerome-Herbinet]

@szaimen (following our conversation a few minutes ago) I've closed my old messy PR (#33669) and I've created this brand new one, with exactly the same changes. It of course takes in account the design feedback of @jancborchardt. I think that everything should be OK (hoping it's not too late for Nextcloud 25).

Thank you! Yes, the PR looks good to me. However I fear we need the compiled js. Can you please run npm run build and commit the changes to this branch?

OK @szaimen, it's done.

[szaimen]

OK @szaimen, it's done.

Thanks! However a npm run sass and commiting the changed files is unfortunately needed as well. Afterwarsa the node test should be green.

[Jerome-Herbinet]

OK @szaimen, it's done.

OK @szaimen, it's done for this as well :-)

[szaimen]

Thanks! I'll review and test it out later on!

[szaimen]

This is how it looks now:

[szaimen]

CI failure unrelated

[szaimen]

Thanks for you patience and effort @Jerome-Herbinet !