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

[Backport] Use proper variables for tooltip styles on tablet devices #2

Closed
wants to merge 2,195 commits into from

Conversation

ansari-ziyaurrahman
Copy link
Owner

Original Pull Request

magento#22258

Description

There are variables available for tooltip customization. This commit fixes an issue
on tablet devices when trying to use them.

Manual testing scenarios

  1. Open Magento/blank/Magento_Checkout/web/css/source/module/checkout/_tooltip.less file and change the values of the tooltip variables as follows (Usually we do this via theme, but it's for testing purpose only):

    @checkout-tooltip-content__background-color: #fff;
    @checkout-tooltip-content__border-color: #f2f5f7;
    @checkout-tooltip-content__active__border-color: @checkout-tooltip-content__border-color;
  2. Navigate to checkout and chec the new styles of field tooltips

  3. Set the width to 768px and check them again:

    Without patch With patch
    image image

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

viktorpetryk and others added 30 commits February 14, 2019 09:36
…o#20529

 - Merge Pull Request magento#20529 from amol2jcommerce/magento2:Fixed-Widget-left-navigation-block-2.2
 - Merged commits:
   1. 7c4cea2
   2. 8a28cca
   3. 5af1479
…word forgot password page wrong css used to make it vertially aling middle magento#21162

 - Merge Pull Request magento#21162 from mage2pratik/magento2:2.2-develop-PR-port-20914
 - Merged commits:
   1. 2220707
   2. be92e7b
   3. bce16b4
…Description are input field in product form while they are defined as textarea magento#21199
…word forgot password page wrong css used to make it vertially aling middle magento#21162
[pangolin] MQE-1430: [2.2.x] Limit allowed bin/magento commands to specific list (MTF)
… customer to the Recent Orders magento#21224

 - Merge Pull Request magento#21224 from eduard13/magento2:2.2-develop-PR-port-21100
 - Merged commits:
   1. 67ef38c
magento-cicd2 and others added 20 commits February 25, 2019 08:04
MAGETWO-96001: Update FedEx Shipping Dates behavior in Tracking Popup
 - merged latest code from mainline branch
Accepted Public Pull Requests:
 - magento#20866: [Backport] Issue magento#20380 fixed for 2.2 (by @maheshWebkul721)


Fixed GitHub Issues:
 - magento#20380: Get Shipping Method as object from order instance gives undefined index. (reported by @maheshWebkul721) has been fixed in magento#20866 by @maheshWebkul721 in 2.2-develop branch
   Related commits:
     1. c495f31
# Conflicts:
#	app/code/Magento/Catalog/Test/Mftf/Data/ProductData.xml
…ibute programmatically magento#21437

 - Merge Pull Request magento#21437 from Nazar65/magento2:backport-issue-19983
 - Merged commits:
   1. b635e09
   2. 97b8f5f
…to#21423

 - Merge Pull Request magento#21423 from eduard13/magento2:2.2-develop-PR-port-21020
 - Merged commits:
   1. abc3e7c
   2. 4b2c1b4
   3. 730987f
Accepted Public Pull Requests:
 - magento#21423: [Backport] Make the module list more deterministic (by @eduard13)
 - magento#21437: [Backport]-issue-195196 Can't upload customer Image attribute programmatically (by @Nazar65)


Fixed GitHub Issues:
 - magento#8479: Sequence of module load order should be deterministic (reported by @scottsb) has been fixed in magento#21423 by @eduard13 in 2.2-develop branch
   Related commits:
     1. abc3e7c
     2. 4b2c1b4
     3. 730987f

 - magento#16116: Modules sort order in config.php is being inconsistent when no changes being made (reported by @comdiler) has been fixed in magento#21423 by @eduard13 in 2.2-develop branch
   Related commits:
     1. abc3e7c
     2. 4b2c1b4
     3. 730987f

 - magento#19983: Can't work customer Image attribute programmatically (reported by @cygnetampatel) has been fixed in magento#21437 by @Nazar65 in 2.2-develop branch
   Related commits:
     1. b635e09
     2. 97b8f5f
Added Squiz.Operators.ValidLogicalOperators rule
ansari-ziyaurrahman pushed a commit that referenced this pull request Apr 16, 2019
@ansari-ziyaurrahman ansari-ziyaurrahman force-pushed the 2.2-develop-PR-port-22258 branch 2 times, most recently from a452da4 to 8c1db6c Compare April 16, 2019 11:31
@ansari-ziyaurrahman ansari-ziyaurrahman force-pushed the 2.2-develop-PR-port-22258 branch from 8c1db6c to cb910d6 Compare April 16, 2019 11:40
@ansari-ziyaurrahman
Copy link
Owner Author

There are more commites in this PR, So I am close and create a new one.

@ansari-ziyaurrahman ansari-ziyaurrahman deleted the 2.2-develop-PR-port-22258 branch April 16, 2019 12:06
@ansari-ziyaurrahman ansari-ziyaurrahman restored the 2.2-develop-PR-port-22258 branch April 16, 2019 12:34
@ansari-ziyaurrahman ansari-ziyaurrahman deleted the 2.2-develop-PR-port-22258 branch April 16, 2019 12:59
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.