Merge pull request #281 from magento-frontend/pub-pr

[Frontend] Public PRs

Author: Yaroslav Onischenko

app/code/Magento/Catalog/view/frontend/templates/product/view/form.phtml

@@ -40,7 +40,7 @@
 <script>
     require([
         'jquery',
-        'Magento_Catalog/js/price-box'
+        'priceBox'
     ], function($){
         var priceBoxes = $('[data-role=priceBox]');
 

app/code/Magento/Ui/view/base/web/js/grid/filters/range.js

@@ -6,8 +6,9 @@ define([
     'underscore',
     'uiLayout',
     'mageUtils',
-    'Magento_Ui/js/form/components/group'
-], function (_, layout, utils, Group) {
+    'Magento_Ui/js/form/components/group',
+    'mage/translate'
+], function (_, layout, utils, Group, $t) {
     'use strict';
 
     return Group.extend({
@@ -29,11 +30,11 @@ define([
                 },
                 ranges: {
                     from: {
-                        label: 'from',
+                        label: $t('from'),
                         dataScope: 'from'
                     },
                     to: {
-                        label: 'to',
+                        label: $t('to'),
                         dataScope: 'to'
                     }
                 }

lib/internal/Magento/Framework/View/Element/Template.php

@@ -139,7 +139,7 @@ public function __construct(Template\Context $context, array $data = [])
     }
 
     /**
-     * Set template context. Sets the object that should represent $this in template
+     * Set template context. Sets the object that should represent $block in template
      *
      * @param \Magento\Framework\View\Element\BlockInterface $templateContext
      * @return void

lib/web/mage/backend/validation.js

@@ -224,7 +224,7 @@
                 }
                 return true;
             },
-            'Please enter a number greater 0 in this field.'
+            $.mage.__('Please enter a number greater 0 in this field.')
         ],
         'validate-rating': [
             function () {
@@ -236,7 +236,7 @@
                 });
                 return noError;
             },
-            'Please select one of each ratings above.'
+            $.mage.__('Please select one of each ratings above.')
         ],
         'validate-downloadable-file': [
             function (v, element) {
@@ -267,7 +267,7 @@
                 }
                 return true;
             },
-            'Please specify Url.'
+            $.mage.__('Please specify Url.')
         ]
     }, function (rule, i) {
         rule.unshift(i);

lib/web/mage/menu.js

@@ -275,6 +275,9 @@ define([
                     if (!target.hasClass('level-top') || !target.has(".ui-menu").length) {
                         window.location.href = target.find('> a').attr('href');
                     }
+                },
+                "click .ui-menu-item:has(.ui-state-active)": function (event) {
+                    this.collapseAll(event, true);
                 }
             });
 
@@ -390,6 +393,39 @@ define([
             };
 
             return setTimeout(handlerProxy, delay || 0);
+        },
+        expand: function( event ) {
+            var newItem = this.active &&
+                this.active
+                    .children( ".ui-menu " )
+                    .children( ".ui-menu-item" )
+                    .first();
+
+            if ( newItem && newItem.length ) {
+                if (newItem.closest( ".ui-menu" ).is( ":visible" )
+                    && newItem.closest( ".ui-menu" ).has( ".all-categories" )
+                ) {
+                    return;
+                }
+
+                this._open( newItem.parent() );
+
+                // Delay so Firefox will not hide activedescendant change in expanding submenu from AT
+                this._delay(function() {
+                    this.focus( event, newItem );
+                });
+            }
+        },
+        select: function( event ) {
+            this.active = this.active || $( event.target ).closest( ".ui-menu-item" );
+            if (this.active.is( ".all-category" )) {
+                this.active = $( event.target ).closest( ".ui-menu-item" );
+            }
+            var ui = { item: this.active };
+            if ( !this.active.has( ".ui-menu" ).length ) {
+                this.collapseAll( event, true );
+            }
+            this._trigger( "select", event, ui );
         }
     });