Skip to content

Commit

Permalink
🔃 [EngCom] Public Pull Requests - 2.2-develop
Browse files Browse the repository at this point in the history
Accepted Public Pull Requests:
 - #20457: [Backport] Area Frontend: Fixed checkbox alignment account information page. (by @suryakant-krish)
 - #20455: [Backport] Fixed 19800 Contact us : design improvement (by @suryakant-krish)
 - #20456: [Backport] Fixed 19791: Logo vertical misalignment.  (by @suryakant-krish)
 - #20284: [Backport] Fix issue causing attribute not loading when using getList (by @GovindaSharma)


Fixed GitHub Issues:
 - #19645: Area Frontend: Account information page checkbox alignment issue. (reported by @suryakant-krish) has been fixed in #20457 by @suryakant-krish in 2.2-develop branch
   Related commits:
     1. 4cdc2f0

 - #19800: Contact us : design improvement (reported by @suryakant-krish) has been fixed in #20455 by @suryakant-krish in 2.2-develop branch
   Related commits:
     1. e8dc461
     2. 3fcc2eb
     3. c1432e6
     4. c57fb36
     5. 6f3207f
     6. ab99ef7
     7. 0641711

 - #19791: Logo vertical misalignment. (reported by @suryakant-krish) has been fixed in #20456 by @suryakant-krish in 2.2-develop branch
   Related commits:
     1. 7b235de

 - #17759: M2.2.5 : CustomerRepository::getList() does not load custom attribute if the name is "company" (reported by @kanduvisla) has been fixed in #20284 by @GovindaSharma in 2.2-develop branch
   Related commits:
     1. 0600244
     2. a59c0b4
  • Loading branch information
magento-engcom-team authored Jan 27, 2019
2 parents 78182ac + f57e0b1 commit 722b213
Show file tree
Hide file tree
Showing 6 changed files with 49 additions and 5 deletions.
2 changes: 1 addition & 1 deletion app/code/Magento/Backend/Model/Search/Customer.php
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ public function load()

$this->searchCriteriaBuilder->setCurrentPage($this->getStart());
$this->searchCriteriaBuilder->setPageSize($this->getLimit());
$searchFields = ['firstname', 'lastname', 'company'];
$searchFields = ['firstname', 'lastname', 'billing_company'];
$filters = [];
foreach ($searchFields as $field) {
$filters[] = $this->filterBuilder
Expand Down
42 changes: 42 additions & 0 deletions app/code/Magento/Contact/view/frontend/web/css/source/_module.less
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
/**
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/

& when (@media-common = true) {
.contact-index-index {
.column:not(.sidebar-main) {
.form.contact {
float: none;
width: 50%;
}
}

.column:not(.sidebar-additional) {
.form.contact {
float: none;
width: 50%;
}
}
}
}

// Mobile
.media-width(@extremum, @break) when (@extremum = 'max') and (@break = @screen__m) {
.contact-index-index {
.column:not(.sidebar-main) {
.form.contact {
float: none;
width: 100%;
}
}

.column:not(.sidebar-additional) {
.form.contact {
float: none;
width: 100%;
}
}
}
}

Original file line number Diff line number Diff line change
Expand Up @@ -365,7 +365,7 @@ public function getList(SearchCriteriaInterface $searchCriteria)
->joinAttribute('billing_telephone', 'customer_address/telephone', 'default_billing', null, 'left')
->joinAttribute('billing_region', 'customer_address/region', 'default_billing', null, 'left')
->joinAttribute('billing_country_id', 'customer_address/country_id', 'default_billing', null, 'left')
->joinAttribute('company', 'customer_address/company', 'default_billing', null, 'left');
->joinAttribute('billing_company', 'customer_address/company', 'default_billing', null, 'left');

$this->collectionProcessor->process($searchCriteria, $collection);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,7 @@ public function testGetList()
->willReturnSelf();
$collection->expects($this->at(7))
->method('joinAttribute')
->with('company', 'customer_address/company', 'default_billing', null, 'left')
->with('billing_company', 'customer_address/company', 'default_billing', null, 'left')
->willReturnSelf();
$this->collectionProcessorMock->expects($this->once())
->method('process')
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@

@menu__background-color: @color-very-dark-grayish-orange;

@menu-logo__padding-bottom: 2.2rem;
@menu-logo__padding-bottom: 1.7rem;
@menu-logo__outer-size: @menu-logo__padding-top + @menu-logo-img__height + @menu-logo__padding-bottom;
@menu-logo__padding-top: 1.2rem;
@menu-logo__padding-top: 1.7rem;
@menu-logo-img__height: 4.1rem;
@menu-logo-img__width: 3.5rem;

Expand Down
2 changes: 2 additions & 0 deletions lib/web/css/source/lib/_forms.less
Original file line number Diff line number Diff line change
Expand Up @@ -300,6 +300,8 @@

input[type="checkbox"] {
.lib-form-element-choice(@_type: input-checkbox);
position: relative;
top: 2px;
}

input[type="radio"] {
Expand Down

0 comments on commit 722b213

Please sign in to comment.