mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

correct delete-icon to icon-delete, fix #11128

This commit is contained in:
Jan-Christoph Borchardt 2014-09-22 18:17:33 +02:00
parent 3c5ac21122
commit 91d9d45c6c
7 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
apps/files/js/fileactions.js
View File

@ -304,7 +304,7 @@
if (img.call) {
img = img(file);
}
html = '<a href="#" original-title="' + escapeHTML(deleteTitle) + '" class="action delete delete-icon" />';
html = '<a href="#" original-title="' + escapeHTML(deleteTitle) + '" class="action delete icon-delete" />';
var element = $(html);
element.data('action', actions['Delete']);
element.on('click', {a: null, elem: parent, actionFunc: actions['Delete'].action}, actionHandler);

6
apps/files/js/filelist.js
View File

@ -1427,7 +1427,7 @@
if (files) {
for (var i=0; i<files.length; i++) {
var deleteAction = this.findFileEl(files[i]).children("td.date").children(".action.delete");
deleteAction.removeClass('delete-icon').addClass('progress-icon');
deleteAction.removeClass('icon-delete').addClass('progress-icon');
}
}
// Finish any existing actions
@ -1445,7 +1445,7 @@
// no files passed, delete all in current dir
params.allfiles = true;
// show spinner for all files
this.$fileList.find('tr>td.date .action.delete').removeClass('delete-icon').addClass('progress-icon');
this.$fileList.find('tr>td.date .action.delete').removeClass('icon-delete').addClass('progress-icon');
}
$.post(OC.filePath('files', 'ajax', 'delete.php'),
@ -1489,7 +1489,7 @@
else {
$.each(files,function(index,file) {
var deleteAction = self.findFileEl(file).find('.action.delete');
deleteAction.removeClass('progress-icon').addClass('delete-icon');
deleteAction.removeClass('progress-icon').addClass('icon-delete');
});
}
}

6
apps/files/tests/js/filelistSpec.js
View File

@ -421,15 +421,15 @@ describe('OCA.Files.FileList tests', function() {
fileList.setFiles(testFiles);
doDelete();
expect(fileList.findFileEl('One.txt').find('.progress-icon:not(.delete-icon)').length).toEqual(1);
expect(fileList.findFileEl('Three.pdf').find('.delete-icon:not(.progress-icon)').length).toEqual(1);
expect(fileList.findFileEl('One.txt').find('.progress-icon:not(.icon-delete)').length).toEqual(1);
expect(fileList.findFileEl('Three.pdf').find('.icon-delete:not(.progress-icon)').length).toEqual(1);
});
it('shows spinner on all files when deleting all', function() {
fileList.setFiles(testFiles);
fileList.do_delete();
expect(fileList.$fileList.find('tr .progress-icon:not(.delete-icon)').length).toEqual(4);
expect(fileList.$fileList.find('tr .progress-icon:not(.icon-delete)').length).toEqual(4);
});
it('updates summary when deleting last file', function() {
var $summary;

4
apps/files_trashbin/js/app.js
View File

@ -41,7 +41,7 @@ OCA.Trashbin.App = {
var fileList = context.fileList;
var tr = fileList.findFileEl(filename);
var deleteAction = tr.children("td.date").children(".action.delete");
deleteAction.removeClass('delete-icon').addClass('progress-icon');
deleteAction.removeClass('icon-delete').addClass('progress-icon');
fileList.disableActions();
$.post(OC.filePath('files_trashbin', 'ajax', 'undelete.php'), {
files: JSON.stringify([filename]),
@ -58,7 +58,7 @@ OCA.Trashbin.App = {
$('.tipsy').remove();
var tr = fileList.findFileEl(filename);
var deleteAction = tr.children("td.date").children(".action.delete");
deleteAction.removeClass('delete-icon').addClass('progress-icon');
deleteAction.removeClass('icon-delete').addClass('progress-icon');
fileList.disableActions();
$.post(OC.filePath('files_trashbin', 'ajax', 'delete.php'), {
files: JSON.stringify([filename]),

4
apps/files_trashbin/js/filelist.js
View File

@ -148,7 +148,7 @@
files = _.pluck(this.getSelectedFiles(), 'name');
for (var i = 0; i < files.length; i++) {
var deleteAction = this.findFileEl(files[i]).children("td.date").children(".action.delete");
deleteAction.removeClass('delete-icon').addClass('progress-icon');
deleteAction.removeClass('icon-delete').addClass('progress-icon');
}
params = {
files: JSON.stringify(files),
@ -202,7 +202,7 @@
else {
for (var i = 0; i < files.length; i++) {
var deleteAction = this.findFileEl(files[i]).children("td.date").children(".action.delete");
deleteAction.removeClass('delete-icon').addClass('progress-icon');
deleteAction.removeClass('icon-delete').addClass('progress-icon');
}
}

4
core/css/fixes.css
View File

@ -15,8 +15,8 @@ select {
font-family: Frutiger, Calibri, 'Myriad Pro', Myriad, Arial, sans-serif;
}
.lte8 .delete-icon { background-image: url('../img/actions/delete.png'); }
.lte8 .delete-icon:hover, .delete-icon:focus {
.lte8 .icon-delete { background-image: url('../img/actions/delete.png'); }
.lte8 .icon-delete:hover, .icon-delete:focus {
background-image: url('../img/actions/delete-hover.png');
}

2
tests/acceptance/tests/pages/files.page.js
View File

@ -78,7 +78,7 @@
};
FilesPage.prototype.deleteButtonId = function(fileName) {
return by.css("tr[data-file='" + fileName + "'] .action.delete.delete-icon");
return by.css("tr[data-file='" + fileName + "'] .action.delete.icon-delete");
};
//================ SHARED ===============================================//