Skip to content

WooCommerce Style Fix#701

Merged
Thomas-A-Reinert merged 3 commits intounderstrap:masterfrom
ZacharyElkins:WooCommerce_styleFix
Jun 8, 2018
Merged

WooCommerce Style Fix#701
Thomas-A-Reinert merged 3 commits intounderstrap:masterfrom
ZacharyElkins:WooCommerce_styleFix

Conversation

@ZacharyElkins
Copy link
Copy Markdown
Collaborator

@ZacharyElkins ZacharyElkins commented Jun 6, 2018

This adds a new SCSS file for WooCommerce. This way if anyone is not using WooCommerce they can easily remove it from their theme.

It also incorporates a fix for the wrapper woocommerce-input-wrapper that was added to billing fields. It's either this or we remove the wrapper.

Reference:

#698

Notes:

  1. gulp styles changed one item from transparent to rgba(0, 0, 0, 0). I'm not sure why this occurred, but they are equivalent values. If anyone has information as to why, I would appreciate it.
    The exact changes were:
    -webkit-tap-highlight-color: transparent; }
    -webkit-tap-highlight-color: rgba(0, 0, 0, 0); }
    Seems like it has to do with auto-prefix.

Adds an scss file for any WooCommerce style issues that may arrise. This will allow users to comment out the style sheet if they are not using WooCommerce.
@0dp
Copy link
Copy Markdown
Collaborator

0dp commented Jun 7, 2018

Interesting with the conversion from transparent to rgba. Will definitely investigate

@Thomas-A-Reinert
Copy link
Copy Markdown
Contributor

Guess why? If ANY browser has major problems with any standards which is that? The browser from hell: SAFARI.

@Thomas-A-Reinert Thomas-A-Reinert merged commit ba5d94d into understrap:master Jun 8, 2018
@ZacharyElkins ZacharyElkins deleted the WooCommerce_styleFix branch June 8, 2018 00:47
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