Patrick Fulton

Results 126 comments of Patrick Fulton

> @pfulton To complete this PR, may I please follow up on these issues described in my previous comment above? > > **Failure pattern 3: Unexpected** Question: Is it safe...

I've updated this to use the graduated version of the dependency. This is now ready for a full review.

@yosevu Mind taking a look at this one?

Given @yosevu's comments above, I've merged this into Spectrum CSS `main` and released it: `@spectrum-css/[email protected]` This also uses `@spectrum-css/[email protected]`, so we're good with the recent updating of the dependencies here.

Looks like there are a few things related to `margin` that I need to check on with this one.

@Westbrook I think the missing thing here is the application of the t-shirt sizes. I tried taking a shot at applying them, but I think I'm missing something. The component...

@Westbrook let me know if these new stories will suffice for now. Also, if I did anything silly, let me know!

Also, I just graduated this component from `beta` over in the CSS repo. If you'd like me to switch to that release, let me know and I can make that...

Pulling in @Westbrook to be sure that he's aware of the things that will be changing here.

Rebased against the latest code in `main` and released the beta: `@spectrum-css/[email protected]`