Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.

Conversation

aveline
Copy link
Contributor

@aveline aveline commented Jan 5, 2023

WHY are these changes introduced?

Supports #7960

WHAT is this pull request doing?

Adds outline prop to Box

@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2023

size-limit report 📦

Path Size
polaris-react-cjs 211.54 KB (+0.01% 🔺)
polaris-react-esm 136.64 KB (+0.02% 🔺)
polaris-react-esnext 191.89 KB (+0.02% 🔺)
polaris-react-css 41.77 KB (+0.02% 🔺)

@aveline aveline marked this pull request as ready for review January 5, 2023 20:19
@aveline aveline requested review from chazdean and kyledurand January 5, 2023 20:19
@kyledurand
Copy link
Member

Anymore context you can add here? I can see it's used on the wrapper outline: var(--p-border-width-1) solid transparent; but I'm unsure why the popover needs this transparent outline. I have a feeling it could be about :focus and if so I'm wondering if it can be handled in the new focus utility that I think someone is working on

@aveline
Copy link
Contributor Author

aveline commented Jan 5, 2023

Yes good question. I don't think it has anything to do with :focus. Digging into it, looks like it has to do with contrast f36c237#diff-0ec08149120c13d1927ed66683c831609974cfa2936fae8933770bf001d0b70a

Copy link
Member

@kyledurand kyledurand 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 digging into the context!

@aveline aveline merged commit e8f74f4 into main Jan 6, 2023
@aveline aveline deleted the layout-box-outline branch January 6, 2023 16:38
kyledurand pushed a commit that referenced this pull request Jan 6, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris@10.18.0

### Minor Changes

- [#7992](#7992)
[`e8f74f4cd`](e8f74f4)
Thanks [@aveline](https://github.com/aveline)! - Added support for
`outline` to `Box`

### Patch Changes

- [#7988](#7988)
[`382784f4e`](382784f)
Thanks [@kyledurand](https://github.com/kyledurand)! - Reduced spacing
on ChoiceList children


- [#7899](#7899)
[`930f077eb`](930f077)
Thanks [@jeradg](https://github.com/jeradg)! - Fixed a bug where
Tooltips nested in Scrollable containers sometimes don't update their
positions correctly


- [#7831](#7831)
[`47487ee0c`](47487ee)
Thanks [@acmertz](https://github.com/acmertz)! - Updated the focus
helper functions to no longer treat buttons with `aria-disabled="true"`
and `tabindex="-1" (but no`disabled\` attribute) as focusable.

## @shopify/plugin-polaris@0.0.25



## polaris.shopify.com@0.28.2

### Patch Changes

- Updated dependencies
\[[`382784f4e`](382784f),
[`930f077eb`](930f077),
[`47487ee0c`](47487ee),
[`e8f74f4cd`](e8f74f4)]:
    -   @shopify/polaris@10.18.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@gwyneplaine gwyneplaine mentioned this pull request Feb 14, 2023
juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
### WHY are these changes introduced?

Supports Shopify#7960 

### WHAT is this pull request doing?

Adds `outline` prop to `Box`
juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris@10.18.0

### Minor Changes

- [Shopify#7992](Shopify#7992)
[`e8f74f4cd`](Shopify@e8f74f4)
Thanks [@aveline](https://github.com/aveline)! - Added support for
`outline` to `Box`

### Patch Changes

- [Shopify#7988](Shopify#7988)
[`382784f4e`](Shopify@382784f)
Thanks [@kyledurand](https://github.com/kyledurand)! - Reduced spacing
on ChoiceList children


- [Shopify#7899](Shopify#7899)
[`930f077eb`](Shopify@930f077)
Thanks [@jeradg](https://github.com/jeradg)! - Fixed a bug where
Tooltips nested in Scrollable containers sometimes don't update their
positions correctly


- [Shopify#7831](Shopify#7831)
[`47487ee0c`](Shopify@47487ee)
Thanks [@acmertz](https://github.com/acmertz)! - Updated the focus
helper functions to no longer treat buttons with `aria-disabled="true"`
and `tabindex="-1" (but no`disabled\` attribute) as focusable.

## @shopify/plugin-polaris@0.0.25



## polaris.shopify.com@0.28.2

### Patch Changes

- Updated dependencies
\[[`382784f4e`](Shopify@382784f),
[`930f077eb`](Shopify@930f077),
[`47487ee0c`](Shopify@47487ee),
[`e8f74f4cd`](Shopify@e8f74f4)]:
    -   @shopify/polaris@10.18.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants