-
Notifications
You must be signed in to change notification settings - Fork 14
Feat/improve UI/UX #210
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
base: main
Are you sure you want to change the base?
Feat/improve UI/UX #210
Conversation
Signed-off-by: torinvdb <[email protected]>
Signed-off-by: torinvdb <[email protected]>
Signed-off-by: torinvdb <[email protected]>
Signed-off-by: torinvdb <[email protected]>
Signed-off-by: torinvdb <[email protected]>
Signed-off-by: torinvdb <[email protected]>
Though I like the present view as well, I believe this improve upon it and makes it much easier to search and find what you are looking for. So I am +1 however I still need to test the code implements the video |
Visually I like this, but I am concerned that it feels a bit complicated, do we really need such a sophisticated filtering system for the quantity of risks / controls we have? |
@ColinEberhardt is the quantity going to remain fixed, though? I was under the assumption we would likely expand and/or update this list as new research, findings, developments, etc in the industry emerged. My main goal with this was to reduce friction to find what people are looking for since it wasn't super intuitive when I was first looking through things on the card view. |
Hi @torinvdb - many thanks for this contribution. We discussed it yesterday in our fortnightly meeting: (which you are of course most welcome to join in future, anyone in FINOS can attend). We discussed for ~20 minutes, and it was very positive. I'll try to summarise here:
So in summary, very positive, and a few small tweaks requested:
We also decided that we don't need a full group review again, so can now fast track. Thank you :-) |
@ColinEberhardt understood - I'll go ahead and make the suggested changes 👍 |
Signed-off-by: torinvdb <[email protected]>
I just made the updates you requested @ColinEberhardt
improved-ui-ux.mov |
Addresses #209
Here's a short video showing the changes I made to improve the UI/UX:
ui-ux.mov