Skip to content

Commit b38ed8b

Browse files
author
Dmitry Kologrivov
committed
Merge branch 'ME-MAGETWO-QA' into 'master3'
ME-MAGETWO-QA Edit warning failed image upload style See merge request !151
2 parents c4fb3ac + 04c500f commit b38ed8b

File tree

3 files changed

+39
-6
lines changed

3 files changed

+39
-6
lines changed

app/code/Magento/ProductVideo/view/adminhtml/web/js/get-video-information.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ require([
403403
errorsMessage.push(tmpError.message);
404404
}
405405

406-
return 'Video cannot be shown due to the following reason: ' + $.unique(errorsMessage).join(', ');
406+
return 'Video cant be shown due to the following reason: ' + $.unique(errorsMessage).join(', ');
407407
};
408408

409409
if (data.error && data.error.code === 400) {

app/code/Magento/ProductVideo/view/adminhtml/web/js/new-video-dialog.js

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,13 @@
44
*/
55
define([
66
'jquery',
7-
'Magento_Ui/js/modal/alert',
87
'jquery/ui',
98
'Magento_Ui/js/modal/modal',
109
'mage/translate',
1110
'mage/backend/tree-suggest',
1211
'mage/backend/validation',
1312
'Magento_ProductVideo/js/get-video-information'
14-
], function ($, alert) {
13+
], function ($) {
1514
'use strict';
1615

1716
$.widget('mage.createVideoPlayer', {
@@ -527,10 +526,13 @@ define([
527526
_onImageLoaded: function (result, file, oldFile, callback) {
528527
var data = JSON.parse(result);
529528

529+
if ($('#video_url').parent().find('.image-upload-error').length > 0) {
530+
$('.image-upload-error').remove();
531+
}
532+
530533
if (data.errorcode || data.error) {
531-
alert({
532-
content: data.error
533-
});
534+
$('#video_url').parent().append('<div class="image-upload-error">' +
535+
'<div class="image-upload-error-cross"></div><span>' + data.error + '</span></div>');
534536

535537
return;
536538
}

app/design/adminhtml/Magento/backend/Magento_ProductVideo/web/css/source/_module.less

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -234,3 +234,34 @@
234234
.image.base-image:hover .image-label {
235235
display: none;
236236
}
237+
238+
.image-upload-error {
239+
background: #F9E1E1;
240+
color: @color-gray19;
241+
font-size: 12px;
242+
font-weight: 400;
243+
padding: 7px 7px 7px 35px;
244+
position: relative;
245+
.image-upload-error-cross {
246+
position: absolute;
247+
left: 8px;
248+
top: 9px;
249+
width: 20px;
250+
height: 20px;
251+
&:before,
252+
&:after {
253+
position: absolute;
254+
left: 8px;
255+
content: ' ';
256+
height: 20px;
257+
width: 4px;
258+
background-color: @color-red9;
259+
}
260+
&:before {
261+
transform: rotate(45deg);
262+
}
263+
&:after {
264+
transform: rotate(-45deg);
265+
}
266+
}
267+
}

0 commit comments

Comments
 (0)