Skip to content

Conversation

3em
Copy link
Collaborator

@3em 3em commented Aug 29, 2025

  • Refactored the StackedList component
  • Added the StackedList page to the Portal - preview
image

Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for harness-xd-review ready!

Name Link
🔨 Latest commit 0984e7d
🔍 Latest deploy log https://app.netlify.com/projects/harness-xd-review/deploys/68b56df52bb0c0000808a4a7
😎 Deploy Preview https://deploy-preview-2108--harness-xd-review.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for harness-design ready!

Name Link
🔨 Latest commit 0984e7d
🔍 Latest deploy log https://app.netlify.com/projects/harness-design/deploys/68b56df516e24b0008eb71bb
😎 Deploy Preview https://deploy-preview-2108--harness-design.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@3em 3em requested a review from praneshg239 August 29, 2025 13:23
@3em 3em force-pushed the ds-migration-stacked-list branch 2 times, most recently from 63c8551 to 788179a Compare September 1, 2025 08:27
@3em 3em force-pushed the ds-migration-stacked-list branch from 3652f6d to 3fcdf8b Compare September 1, 2025 08:45
@praneshg239
Copy link
Collaborator

For header, can we StackedList.Header instead using StackedList.Item with isHeader prop? It is confusing.

image

@praneshg239
Copy link
Collaborator

shall we remove paddingX or paddingY whose value is md? Because it is the default value.

@3em 3em requested a review from praneshg239 September 1, 2025 10:34
@3em 3em merged commit fc99ecf into main Sep 1, 2025
14 checks passed
@3em 3em deleted the ds-migration-stacked-list branch September 1, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants