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

Überarbeitung Rollenübersicht auf Person #511

Open
RolandStuder opened this issue Apr 3, 2018 · 4 comments
Open

Überarbeitung Rollenübersicht auf Person #511

RolandStuder opened this issue Apr 3, 2018 · 4 comments
Labels

Comments

@RolandStuder
Copy link
Contributor

IST

Die Rollenübersicht auf einer Person ist nicht besonders übersichtlich, gerade wenn eine Person viele Rollen hat. Auch kann eine Zugehörigkeit zu einer Ebene wild verteilt in der Liste auftauchen

image

SOLL

Die Rollen zu einer Ebene werden gruppiert und eingerückt dargestellt. Dies ist so bereits in einem Branch als Vorschlag vorhanden siehe https://github.com/hitobito/hitobito/commits/feature/better-roles-aside:

image

  • Feedback ist willkommen.
  • Auch Codeverbesserungen (es ist sehr viel Logik in der View drin, man müsste wohl das eine oder andere in Helper auslagern).
@RolandStuder RolandStuder added this to Backlog in Aktuelle Arbeiten via automation May 18, 2018
@RolandStuder
Copy link
Contributor Author

I noticed that this does not look very good, if you have only few roles. I think a chip based layout will work better, something like this:

image


Example with more roles:

image

@RolandStuder
Copy link
Contributor Author

RolandStuder commented Apr 12, 2019

Nach vielen Iteration habe ich nun folgendes umgesetzt:

Vorher

image

Nachher:

image

  • Rollen sind gruppiert nach Ebene
  • Nur noch einen Rolle hinzufügen Button pro Ebene
  • Insgesamt übersichtlicher

@kronn
Copy link
Member

kronn commented Feb 28, 2020

Gibt es einen PR dafür?

@RolandStuder
Copy link
Contributor Author

Habe nie einen PR gemacht, aber es gibt einen Branch hierfür: https://github.com/hitobito/hitobito/tree/feature/better-roles-aside

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants