Skip to content

Conversation

@jenny-s51
Copy link
Member

@jenny-s51 jenny-s51 commented Nov 10, 2025

Closes #716 #717

What changed:

  1. isInline prop removed on "Cancel" buttons - no longer showing underline. (Applies "link" variant):
Screenshot 2025-11-10 at 1 33 13 PM Screenshot 2025-11-10 at 1 33 21 PM Screenshot 2025-11-10 at 3 34 10 PM

2) Added `Stack` layout with gutter to resolve spacing issue:

Before:
Screenshot 2025-11-10 at 3 12 57 PM

After:
Screenshot 2025-11-10 at 1 59 22 PM

cc @tobiastal, per your feedback

@github-project-automation github-project-automation bot moved this to Needs Triage in Kubeflow Notebooks Nov 10, 2025
@google-oss-prow google-oss-prow bot added the area/frontend area - related to frontend components label Nov 10, 2025
@google-oss-prow google-oss-prow bot added area/v2 area - version - kubeflow notebooks v2 size/XL labels Nov 10, 2025
@jenny-s51 jenny-s51 force-pushed the cancel-btn-and-expandable-fix branch from 39f4446 to 2a4fe40 Compare November 10, 2025 20:37
@ederign
Copy link
Member

ederign commented Nov 10, 2025

/approve
/lgtm

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ederign

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 Nov 12, 2025

/ok-to-test

@google-oss-prow google-oss-prow bot merged commit 2cc4244 into kubeflow:notebooks-v2 Nov 12, 2025
12 checks passed
@github-project-automation github-project-automation bot moved this from Needs Triage to Done in Kubeflow Notebooks Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved area/frontend area - related to frontend components area/v2 area - version - kubeflow notebooks v2 lgtm ok-to-test size/XL

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants