Rework permission UI

[AdrienClairembault]

Checklist before requesting a review

• I have performed a self-review of my code.
• I have added tests that prove my fix is effective or that my feature works.

Description

Before;

After:

The display is more compact and try to avoid unnecessary informations compared to the previous iteration.
The form is cleaned up as the dropdown were a bit out of place.

One of the main goal was to hide the extra entities id / is recursive options which are not often used and create confusion for the users (we can't drop them because they are still used by few).

They are hidden by default:

And can be toggled with a button:

When the list is empty, we have a specific message:

[AdrienClairembault]

Note: this file should be replaced by proper controllers + templates.
I've added it to the backlog so it can be done separately.

[AdrienClairembault]

Note: this file should be replaced by proper controllers + templates.
I've added it to the backlog so it can be done separately.

[cconard96]

Additional review from functional testing:

Avoid reloading entire page every time user wants to add visibility targets; it is very annoying. It doesn't reload the page when targets are removed.

[AdrienClairembault]

Avoid reloading entire page every time user wants to add visibility targets; it is very annoying. It doesn't reload the page when targets are removed.

I agree but I wanted to keep changes low here as there are already a lot of lines changed.
It is planned as a dedicated task in the backlog: https://github.com/orgs/glpi-project/projects/26/views/1?groupedBy%5BcolumnId%5D=247277380&pane=issue&itemId=175839857.

[AdrienClairembault]

Rebased for conflicts, there was no others changes.