Replies: 2 comments 4 replies
-
|
Looks like a nice addition for sure. I wonder if the pagination should be hidden if there's not many identity providers. In most cases there will never be more than a handful, and it would be strange to show pagination in those cases. @ssilvert FIY |
Beta Was this translation helpful? Give feedback.
4 replies
-
|
@stianst It seems that we left this in an "accepted but not yet merged PR" state 😄 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Motivation:
On keycloak’s account management console, if a realm has a lot of identity providers, the user is presented with a huge list which he has to scroll in order to find the desired identity provider to link his account with, since there is no search/filtering in the UI. Moreover, a lot of information is fetched, making the webpage quite resource demanding in terms of memory and network bandwidth.
Design proposal:
We propose to perform minimal changes on the current design and add a paging and filtering on both {Linked Login Providers, Unlinked Login Providers} lists.
Our enhancement is done by making as few as possible changes on both the react and java sides.
So, with the following minimal changes:
The loading times are hugely improved:
We have ~4300 IdPs in our keycloak and we measured the loading times and the idp-related data fetching.
The pull request can be found here
We also attach two screenshots to showcase the UI changes:
Before:
After:
Beta Was this translation helpful? Give feedback.
All reactions