-
Notifications
You must be signed in to change notification settings - Fork 41
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
Apply MUI theming to Registered Model List View #432
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
66ef76e
to
105a115
Compare
/retest |
/ok-to-test |
@tarilabs can you please fire the workflows here? |
@jenny-s51 are we also able to add a little bit of spacing between the filter button and the text in the image above? |
@@ -7,6 +7,7 @@ import { filterLiveModels } from '~/app/utils'; | |||
import ModelRegistrySelectorNavigator from './ModelRegistrySelectorNavigator'; | |||
import RegisteredModelListView from './RegisteredModels/RegisteredModelListView'; | |||
import { modelRegistryUrl } from './routeUtils'; | |||
import '~/style/MUI-theme.scss'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I might not understand how all of this works, but are we able to just import this at a higher level so we don't need to import in multiple places?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, removed other imports
b810efe
to
3468804
Compare
Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com> adjust padding of label for consistency in focused state fix filter label text, add padding, remove unused imports
3468804
to
1541ea4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reviewing @Griffin-Sullivan !
The filter on the registered models page doesn't change the filter label when selecting Owner. Should say "Find by Owner" here.
Nice catch, fixed this
@jenny-s51 are we also able to add a little bit of spacing between the filter button and the text in the image above?
Yes, added some padding
Description
Applies theming and styles to MR UI for Registered Models List View.
Before:
After:
How Has This Been Tested?
Visual updates only - tested manually in UI.
Merge criteria:
DCO
check)If you have UI changes