Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove mix blend mode on primary button #9912

Merged
merged 2 commits into from
Aug 14, 2023
Merged

Remove mix blend mode on primary button #9912

merged 2 commits into from
Aug 14, 2023

Conversation

sam-b-rose
Copy link
Member

WHY are these changes introduced?

Fixes https://github.com/Shopify/polaris-internal/issues/1035

WHAT is this pull request doing?

The mix-blend-mode was causing a visual regression on Android Chrome.

This was tested on a Google Pixel 6 Pro on Chrome

Before After
before after

This was causing a visual regression on Andriod Chrome
Copy link
Contributor

@jesstelford jesstelford left a comment

Choose a reason for hiding this comment

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

Code looks good other than one stylelint issue and failing CI (possibly related).

@sam-b-rose sam-b-rose merged commit 00b8312 into main Aug 14, 2023
14 checks passed
@sam-b-rose sam-b-rose deleted the android-button-fix branch August 14, 2023 15:13
This was referenced Aug 14, 2023
sam-b-rose pushed a commit that referenced this pull request Aug 15, 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/stylelint-polaris@14.0.0

### Major Changes

- [#9887](#9887)
[`a3da87e2e`](a3da87e)
Thanks [@lgriffee](https://github.com/lgriffee)! - Removed deprecated
v10 custom properties from stylelint-polaris disallowed lists

### Patch Changes

- Updated dependencies
\[[`36e4ee8af`](36e4ee8)]:
    -   @shopify/polaris-tokens@7.5.3

## @shopify/polaris@11.12.0

### Minor Changes

- [#10042](#10042)
[`1d82a3b12`](1d82a3b)
Thanks [@m4thieulavoie](https://github.com/m4thieulavoie)! - introduce a
subdued prop to the popover pane component


- [#9868](#9868)
[`cb1dbbbd3`](cb1dbbb)
Thanks [@mattkubej](https://github.com/mattkubej)! - Introduce
disableKeyboardShortcuts prop to IndexFilters


- [#9862](#9862)
[`93b902094`](93b9020)
Thanks [@brittcorry](https://github.com/brittcorry)! - Added support for
the ```Filters``closeOnChildOverlayClick``` prop to `IndexFilters`


- [#9872](#9872)
[`f585a57e0`](f585a57)
Thanks [@fatimasajadi](https://github.com/fatimasajadi)! - Add a
critical status to the IndexTable

### Patch Changes

- [#10012](#10012)
[`3ae94cef0`](3ae94ce)
Thanks [@melvinadu](https://github.com/melvinadu)! - Fixed wrapping
overflow of strings with no spacing within `Filters` popover


- [#9889](#9889)
[`0cbdbb4f5`](0cbdbb4)
Thanks [@chloerice](https://github.com/chloerice)! - Set the preferred
position of `Pagination` and `Page` `secondaryActions` button tooltips
to `below`


- [#10045](#10045)
[`65ad4e27c`](65ad4e2)
Thanks [@sophschneider](https://github.com/sophschneider)! - Fixed
BannerExperimental no title hidden icon variant


- [#9885](#9885)
[`f0d288099`](f0d2880)
Thanks [@bsharrow](https://github.com/bsharrow)! - `Aligned
the`SkeletonPage\``title`font-weight with the`Page` title


- [#9860](#9860)
[`af0c9d4a2`](af0c9d4)
Thanks [@sophschneider](https://github.com/sophschneider)! - Updated CI
tests to account for both polarisSummerEditions2023 beta flag states


- [#9874](#9874)
[`5569ac69a`](5569ac6)
Thanks [@chloerice](https://github.com/chloerice)! - Fixed `Page` first
`Header` row misalignment


- [#9850](#9850)
[`57d8d5506`](57d8d55)
Thanks [@kyledurand](https://github.com/kyledurand)! - Rebuilt `Filters`
`SearchField`


- [#8988](#8988)
[`535b3fc91`](535b3fc)
Thanks [@m4thieulavoie](https://github.com/m4thieulavoie)! - Added
support for rendering `Text` `as` a `strong` tag


- [#9912](#9912)
[`00b831292`](00b8312)
Thanks [@samrose3](https://github.com/samrose3)! - Fixed primary Button
styles for Chrome on Android devices

- Updated dependencies
\[[`d1bee0f87`](d1bee0f),
[`36e4ee8af`](36e4ee8)]:
    -   @shopify/polaris-icons@7.8.1
    -   @shopify/polaris-tokens@7.5.3

## @shopify/polaris-cli@0.2.30

### Patch Changes

-   Updated dependencies \[]:
    -   @shopify/polaris-migrator@0.19.9

## @shopify/polaris-codemods@0.1.12

### Patch Changes

- Updated dependencies
\[[`a3da87e2e`](a3da87e),
[`36e4ee8af`](36e4ee8)]:
    -   @shopify/stylelint-polaris@14.0.0
    -   @shopify/polaris-tokens@7.5.3

## @shopify/polaris-icons@7.8.1

### Patch Changes

- [#9879](#9879)
[`d1bee0f87`](d1bee0f)
Thanks [@kyledurand](https://github.com/kyledurand)! - Optimized some
unoptimized icons

## @shopify/polaris-migrator@0.19.9

### Patch Changes

- Updated dependencies
\[[`a3da87e2e`](a3da87e),
[`36e4ee8af`](36e4ee8)]:
    -   @shopify/stylelint-polaris@14.0.0
    -   @shopify/polaris-tokens@7.5.3

## @shopify/polaris-tokens@7.5.3

### Patch Changes

- [#9904](#9904)
[`36e4ee8af`](36e4ee8)
Thanks [@benwolfram](https://github.com/benwolfram)! - Added a unit to
the `space-0` token

## polaris.shopify.com@0.57.0

### Minor Changes

- [#9886](#9886)
[`d85099c89`](d85099c)
Thanks [@lgriffee](https://github.com/lgriffee)! - Updated Polaris
Stylelint version matchups chart

### Patch Changes

- Updated dependencies
\[[`d1bee0f87`](d1bee0f),
[`1d82a3b12`](1d82a3b),
[`36e4ee8af`](36e4ee8),
[`cb1dbbbd3`](cb1dbbb),
[`3ae94cef0`](3ae94ce),
[`0cbdbb4f5`](0cbdbb4),
[`93b902094`](93b9020),
[`65ad4e27c`](65ad4e2),
[`f0d288099`](f0d2880),
[`af0c9d4a2`](af0c9d4),
[`5569ac69a`](5569ac6),
[`f585a57e0`](f585a57),
[`57d8d5506`](57d8d55),
[`535b3fc91`](535b3fc),
[`00b831292`](00b8312)]:
    -   @shopify/polaris-icons@7.8.1
    -   @shopify/polaris@11.12.0
    -   @shopify/polaris-tokens@7.5.3

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
### WHY are these changes introduced?

Fixes Shopify/polaris-internal#1035

### WHAT is this pull request doing?

The `mix-blend-mode` was causing a visual regression on Android Chrome.

This was tested on a Google Pixel 6 Pro on Chrome

| Before | After |
| --- | --- |
| <img width="516" alt="before"
src="https://github.com/Shopify/polaris/assets/11774595/3387fee2-7b53-4dec-89b4-4cdd1ee7465a">
| <img width="516" alt="after"
src="https://github.com/Shopify/polaris/assets/11774595/358f61ea-c31a-4d95-bf73-c1b9e7c5db30">
|

---------

Co-authored-by: Sophie Schneider <thesophieschneider@gmail.com>
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.

3 participants