Pwa-studio: [Styles]: Close button displayed incorrect for cart

Created on 29 Nov 2018  路  4Comments  路  Source: magento/pwa-studio

This issue is for the following packages:

[X] venia-concept
[ ] pwa-buildpack
[ ] peregrine
[ ] pwa-devdocs
[ ] upward-js
[ ] upward-spec

This issue is a:

[X] Bug
[ ] Feature suggestion
[ ] Documentation issue
[ ] Other (Please Specify)

Environment

| Question | Answer |
| --------------------------- | :-----------: |
| Magento version | |
| Operating System + version | |
| node.js version (node -v) | |
| npm version (npm -v) | |

Description

close cart button displayed incorrect as you can see on screenshot below
image

Expected result:

image

Possible solutions:

in class miniCart-header fix styles:

.miniCart-header{
    grid-template-columns: 0.9fr; /* was 1fr */
    justify-content: end; /* remove this style rule */
}

bug venia-concept

Most helpful comment

Hello, I have submitted PR which fixes this issue: https://github.com/magento-research/pwa-studio/pull/600

All 4 comments

Hello, I have submitted PR which fixes this issue: https://github.com/magento-research/pwa-studio/pull/600

Fantastic, thanks @codeAdrian! Every contribution counts.

Update. Production check. this issue was fixed, close button displaying correctly:
image
Reason for close. Fixed

Hello, thanks for confirming the fix. I'm just wondering if we should keep the issue open until PR has been approved and merged? Maybe @ericerway can correct me if I'm wrong. Since issues are synced with Zenhub and used for tracking fixes and features.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

senthil134 picture senthil134  路  7Comments

ericerway picture ericerway  路  6Comments

dambrogia picture dambrogia  路  7Comments

Jordaneisenburger picture Jordaneisenburger  路  3Comments

Starotitorov picture Starotitorov  路  3Comments