Skip to content

Conversation

@jenny-s51
Copy link
Member

@jenny-s51 jenny-s51 commented Jan 17, 2025

Adds MUI support to Notebooks 2.0 UI. Applies theming to Workspaces table.

Before:
Screenshot 2025-01-17 at 11 57 48 AM

After:
Screenshot 2025-01-17 at 11 34 14 AM

Copy link

@paulovmr paulovmr left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @jenny-s51 ! Changes look OK to me, if you could please just run the eslint with npm run test:fix in the workspaces/frontend directory and commit the changes, it will fix the some formatting issues.

…pport

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply env var to activate MUI theme

remove masthead toggle, fix padding

add bottom padding

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply latest changes from MR and npm run test:fix
Copy link
Member 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.

Thank you for your review and feedback @paulovmr.

I've rebased the branch, updated the stylesheet with the latest fixes from Model Registry, resolved the PR conflicts, and ran npm run test:fix.

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ederign, paulovmr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ederign
Copy link
Member

ederign commented Jan 22, 2025

Great to see you contributing @jenny-s51 !

@ederign
Copy link
Member

ederign commented Jan 22, 2025

/lgtm

@google-oss-prow google-oss-prow bot added the lgtm label Jan 22, 2025
@google-oss-prow google-oss-prow bot merged commit 9479c7b into kubeflow:notebooks-v2 Jan 22, 2025
3 checks passed
Mohamed-ben-khemis pushed a commit to Mohamed-ben-khemis/notebooks that referenced this pull request Mar 13, 2025
…pport (kubeflow#183)

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply env var to activate MUI theme

remove masthead toggle, fix padding

add bottom padding

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply latest changes from MR and npm run test:fix
mkoushni pushed a commit to mkoushni/notebooks that referenced this pull request May 28, 2025
…pport (kubeflow#183)

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply env var to activate MUI theme

remove masthead toggle, fix padding

add bottom padding

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply latest changes from MR and npm run test:fix

Signed-off-by: CI Bot <mkoushni@redhat.com>
andyatmiami pushed a commit to andyatmiami/kubeflow-notebooks that referenced this pull request Jun 13, 2025
…pport (kubeflow#183)

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply env var to activate MUI theme

remove masthead toggle, fix padding

add bottom padding

Signed-off-by: Jenny <32821331+jenny-s51@users.noreply.github.com>

apply latest changes from MR and npm run test:fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants