Skip to content

Commit 4ae733a

Browse files
author
Oleksandr Dubovyk
committed
MAGETWO-38963: Storefront "Layered Navigation" menu items' titles are overlapped by "Expand" button
-Merge remote-tracking branch 'origin/develop' into MAGETWO-38963 -Conflicts: - app/design/frontend/Magento/blank/Magento_LayeredNavigation/web/css/source/_module.less - app/design/frontend/Magento/luma/Magento_LayeredNavigation/web/css/source/_module.less
2 parents f30b578 + 02e0378 commit 4ae733a

File tree

2,069 files changed

+27579
-23074
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

2,069 files changed

+27579
-23074
lines changed

app/code/Magento/AdminNotification/Block/System/Messages.php

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -120,11 +120,8 @@ public function getSystemMessageDialogJson()
120120
return $this->jsonHelper->jsonEncode(
121121
[
122122
'systemMessageDialog' => [
123-
'autoOpen' => false,
124-
'width' => '75%',
125-
'modal' => true,
126-
'minHeight' => '0',
127-
'dialogClass' => 'ui-dialog-active ui-popup-message',
123+
'buttons' => [],
124+
'modalClass' => 'ui-dialog-active ui-popup-message modal-system-messages',
128125
'ajaxUrl' => $this->_getMessagesUrl()
129126
],
130127
]

app/code/Magento/AdminNotification/Model/Feed.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ class Feed extends \Magento\Framework\Model\AbstractModel
7272
* @param \Magento\Framework\App\DeploymentConfig $deploymentConfig
7373
* @param \Magento\Framework\App\ProductMetadataInterface $productMetadata
7474
* @param \Magento\Framework\UrlInterface $urlBuilder
75-
* @param \Magento\Framework\Model\ModelResource\AbstractResource $resource
75+
* @param \Magento\Framework\Model\ResourceModel\AbstractResource $resource
7676
* @param \Magento\Framework\Data\Collection\AbstractDb $resourceCollection
7777
* @param array $data
7878
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
@@ -86,7 +86,7 @@ public function __construct(
8686
\Magento\Framework\App\DeploymentConfig $deploymentConfig,
8787
\Magento\Framework\App\ProductMetadataInterface $productMetadata,
8888
\Magento\Framework\UrlInterface $urlBuilder,
89-
\Magento\Framework\Model\ModelResource\AbstractResource $resource = null,
89+
\Magento\Framework\Model\ResourceModel\AbstractResource $resource = null,
9090
\Magento\Framework\Data\Collection\AbstractDb $resourceCollection = null,
9191
array $data = []
9292
) {

app/code/Magento/AdminNotification/Model/ResourceModel/Grid/Collection.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class Collection extends \Magento\AdminNotification\Model\ResourceModel\Inbox\Co
1818
/**
1919
* Add remove filter
2020
*
21-
* @return \Magento\AdminNotification\Model\ResourceModel\Grid\Collection|\Magento\Framework\Model\ModelResource\Db\Collection\AbstractCollection
21+
* @return \Magento\AdminNotification\Model\ResourceModel\Grid\Collection|\Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
2222
*/
2323
protected function _initSelect()
2424
{

app/code/Magento/AdminNotification/Model/ResourceModel/Inbox.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
*
1111
* @author Magento Core Team <core@magentocommerce.com>
1212
*/
13-
class Inbox extends \Magento\Framework\Model\ModelResource\Db\AbstractDb
13+
class Inbox extends \Magento\Framework\Model\ResourceModel\Db\AbstractDb
1414
{
1515
/**
1616
* AdminNotification Resource initialization

app/code/Magento/AdminNotification/Model/ResourceModel/Inbox/Collection.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
*
1111
* @author Magento Core Team <core@magentocommerce.com>
1212
*/
13-
class Collection extends \Magento\Framework\Model\ModelResource\Db\Collection\AbstractCollection
13+
class Collection extends \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
1414
{
1515
/**
1616
* Resource collection initialization

app/code/Magento/AdminNotification/Model/ResourceModel/Inbox/Collection/Critical.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
*/
88
namespace Magento\AdminNotification\Model\ResourceModel\Inbox\Collection;
99

10-
class Critical extends \Magento\Framework\Model\ModelResource\Db\Collection\AbstractCollection
10+
class Critical extends \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
1111
{
1212
/**
1313
* Resource collection initialization

app/code/Magento/AdminNotification/Model/ResourceModel/System/Message.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*/
66
namespace Magento\AdminNotification\Model\ResourceModel\System;
77

8-
class Message extends \Magento\Framework\Model\ModelResource\Db\AbstractDb
8+
class Message extends \Magento\Framework\Model\ResourceModel\Db\AbstractDb
99
{
1010
/**
1111
* Flag that notifies whether Primary key of table is auto-incremeted

app/code/Magento/AdminNotification/Model/ResourceModel/System/Message/Collection.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*/
66
namespace Magento\AdminNotification\Model\ResourceModel\System\Message;
77

8-
class Collection extends \Magento\Framework\Model\ModelResource\Db\Collection\AbstractCollection
8+
class Collection extends \Magento\Framework\Model\ResourceModel\Db\Collection\AbstractCollection
99
{
1010
/**
1111
* System message list
@@ -28,7 +28,7 @@ class Collection extends \Magento\Framework\Model\ModelResource\Db\Collection\Ab
2828
* @param \Magento\Framework\Event\ManagerInterface $eventManager
2929
* @param \Magento\Framework\Notification\MessageList $messageList
3030
* @param mixed $connection
31-
* @param \Magento\Framework\Model\ModelResource\Db\AbstractDb $resource
31+
* @param \Magento\Framework\Model\ResourceModel\Db\AbstractDb $resource
3232
*/
3333
public function __construct(
3434
\Magento\Framework\Data\Collection\EntityFactory $entityFactory,
@@ -37,7 +37,7 @@ public function __construct(
3737
\Magento\Framework\Event\ManagerInterface $eventManager,
3838
\Magento\Framework\Notification\MessageList $messageList,
3939
\Magento\Framework\DB\Adapter\AdapterInterface $connection = null,
40-
\Magento\Framework\Model\ModelResource\Db\AbstractDb $resource = null
40+
\Magento\Framework\Model\ResourceModel\Db\AbstractDb $resource = null
4141
) {
4242
$this->_messageList = $messageList;
4343
parent::__construct($entityFactory, $logger, $fetchStrategy, $eventManager, $connection, $resource);

app/code/Magento/AdminNotification/Model/ResourceModel/System/Message/Collection/Synchronized.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ class Synchronized extends \Magento\AdminNotification\Model\ResourceModel\System
1717
/**
1818
* Store new messages in database and remove outdated messages
1919
*
20-
* @return $this|\Magento\Framework\Model\ModelResource\Db\AbstractDb
20+
* @return $this|\Magento\Framework\Model\ResourceModel\Db\AbstractDb
2121
*/
2222
public function _afterLoad()
2323
{

app/code/Magento/AdminNotification/view/adminhtml/templates/system/messages/popup.phtml

Lines changed: 12 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99
?>
1010
<?php /** @var $block \Magento\AdminNotification\Block\System\Messages\UnreadMessagePopup */ ?>
1111

12-
<div id="system_messages_list" title="<?php echo $block->escapeHtml($block->getPopupTitle()); ?>">
13-
<ul class="message-system-list">
12+
<div style="display:none" id="system_messages_list" data-role="system_messages_list" title="<?php echo $block->escapeHtml($block->getPopupTitle()); ?>">
13+
<ul class="message-system-list messages">
1414
<?php foreach ($block->getUnreadMessages() as $message): ?>
1515
<li class="message message-warning <?php /* @escapeNotVerified */ echo $block->getItemClass($message);?>">
1616
<?php /* @escapeNotVerified */ echo $message->getText();?>
@@ -22,29 +22,17 @@
2222
<script>
2323
require([
2424
"jquery",
25-
"jquery/ui"
25+
"Magento_Ui/js/modal/modal"
2626
], function($){
27-
28-
$(function() {
29-
$("#system_messages_list").dialog({
30-
autoOpen: true,
31-
resizable: false,
32-
width: '75%',
33-
modal: true,
34-
minHeight: '0',
35-
dialogClass: 'ui-popup-message',
36-
position: {
37-
my: 'left+12.5% top',
38-
at: 'center top',
39-
of: 'body'
40-
},
41-
open: function() {
42-
$(this).closest('.ui-dialog').addClass('ui-dialog-active');
43-
44-
var topMargin = $(this).closest('.ui-dialog').children('.ui-dialog-titlebar').outerHeight() + 10;
45-
$(this).closest('.ui-dialog').css('margin-top', topMargin);
46-
}
27+
if (this.modal) {
28+
this.modal.html($('[data-role="system_messages_list"]').html());
29+
} else {
30+
this.modal = $('[data-role="system_messages_list"]').modal({
31+
modalClass: 'modal-system-messages ui-popup-message',
32+
type: 'popup',
33+
buttons: []
4734
});
48-
});
35+
}
36+
this.modal.modal('openModal');
4937
});
5038
</script>

0 commit comments

Comments
 (0)