This repository was archived by the owner on Jul 22, 2025. It is now read-only.
UX: Improve rough edges of AI usage page #1014
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Ensure all text uses I18n * Change from <button> usage to <DButton> * Use <AdminConfigAreaCard> in place of custom card styles * Format numbers nicely using our number format helper, show full values on hover using title attr * Ensure 0 is always shown for counters, instead of being blank
martin-brennan
force-pushed
the
issue/cleanup-usage-page
branch
3 times, most recently
from
4fca0b8 to
afdd13b
Compare
Use ConditionalLoadingSpinner to hide load of usage data, this prevents us hanging on page load with a white screen.
martin-brennan
force-pushed
the
issue/cleanup-usage-page
branch
from
afdd13b to
13c3481
Compare
martin-brennan
force-pushed
the
issue/cleanup-usage-page
branch
from
d12b8cb to
13b7576
Compare
ZogStriP
approved these changes
Dec 11, 2024
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit makes several UX improvements along with code cleanup
for the new AI usage page:
show full values on hover using title attr
and split it into 2 columns when there are > 25 users
Also, it improves the load experience of the page. Previously
we loaded all the report data up front on the server when the
admin visited this usage route which caused a white screen
in some cases and could take quite long.
Now, we show loading spinners for the report and cards below
while we fetch the data.
Note, the users list still needs more work, a reusable avatar + username component
from core. It's an improvement from what it was before, where it pushed the
other stat card down the page because 50 users were shown.
Empty data
Mobile