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

Model Info: Theme "Versions" and "Details" Tabs #3

Closed
wants to merge 2 commits into from

Conversation

jenny-s51
Copy link
Owner

@jenny-s51 jenny-s51 commented Oct 8, 2024

Description

Towards https://issues.redhat.com/browse/RHOAIENG-13191.

Merge criteria:

Dependent on changes in kubeflow#451 - will open for review onced merged/rebased.

@jenny-s51 jenny-s51 marked this pull request as draft October 8, 2024 21:05
Signed-off-by: Jenny <[email protected]>

fix tests

Signed-off-by: Jenny <[email protected]>

add fieldset to custom components

fix spacing issue with helper text

Signed-off-by: Jenny <[email protected]>

remove comment

Signed-off-by: Jenny <[email protected]>

fixes for text area

Signed-off-by: Jenny <[email protected]>

format

Signed-off-by: Jenny <[email protected]>

add model description fix

Signed-off-by: Jenny <[email protected]>

create reusable fieldset component

Signed-off-by: Jenny <[email protected]>

move fieldset to components

Signed-off-by: Jenny <[email protected]>

add prop for className

Signed-off-by: Jenny <[email protected]>

lint

Signed-off-by: Jenny <[email protected]>

fix import warnings

fix TS error with FormFieldset.tsx

Signed-off-by: Jenny <[email protected]>

fix import order errors
@jenny-s51 jenny-s51 force-pushed the themeModelInfoScreens branch 2 times, most recently from 269e414 to 90b231e Compare October 10, 2024 18:05
Signed-off-by: Jenny <[email protected]>

apply theming to model info screens

fix button color on hover

add comment

fix button color on hover

Signed-off-by: Jenny <[email protected]>

format

update to absolute path
@jenny-s51 jenny-s51 force-pushed the themeModelInfoScreens branch from c7e9be9 to 0ce7107 Compare October 10, 2024 19:26
Copy link
Owner Author

@jenny-s51 jenny-s51 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closing in favor of upstream PR.

@jenny-s51 jenny-s51 closed this Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant