Fix whitespace and file action cleanup
Signed-off-by: Julius Härtl <jus@bitgrid.net>
This commit is contained in:
parent
fc99811d0a
commit
78056a3bef
|
@ -362,17 +362,16 @@
|
||||||
// remove previous
|
// remove previous
|
||||||
$tr.find('.action-menu').remove();
|
$tr.find('.action-menu').remove();
|
||||||
|
|
||||||
if (this.actions.length > 0) {
|
var $el = this._renderInlineAction({
|
||||||
var $el = this._renderInlineAction({
|
name: 'menu',
|
||||||
name: 'menu',
|
displayName: '',
|
||||||
displayName: '',
|
iconClass: 'icon-more',
|
||||||
iconClass: 'icon-more',
|
altText: t('files', 'Actions'),
|
||||||
altText: t('files', 'Actions'),
|
action: this._showMenuClosure
|
||||||
action: this._showMenuClosure
|
}, false, context);
|
||||||
}, false, context);
|
|
||||||
|
$el.addClass('permanent');
|
||||||
|
|
||||||
$el.addClass('permanent');
|
|
||||||
}
|
|
||||||
},
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -209,7 +209,7 @@ OCA.Sharing.PublicApp = {
|
||||||
// Remove the link. This means that files without a default action fail hard
|
// Remove the link. This means that files without a default action fail hard
|
||||||
$tr.find('a.name').attr('href', '#');
|
$tr.find('a.name').attr('href', '#');
|
||||||
|
|
||||||
delete this.fileActions.actions.all.Download;
|
this.fileActions.actions.all = {};
|
||||||
}
|
}
|
||||||
return $tr;
|
return $tr;
|
||||||
};
|
};
|
||||||
|
@ -299,6 +299,7 @@ OCA.Sharing.PublicApp = {
|
||||||
|
|
||||||
if (hideDownload === 'true') {
|
if (hideDownload === 'true') {
|
||||||
this.fileList.$el.find('#headerSelection').remove();
|
this.fileList.$el.find('#headerSelection').remove();
|
||||||
|
this.fileList.$el.find('.summary').find('td:first-child').remove();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -114,7 +114,7 @@ describe('OC.Share.ShareDialogLinkShareView', function () {
|
||||||
|
|
||||||
$hideDownloadCheckbox = view.$el.find('.hideDownloadCheckbox');
|
$hideDownloadCheckbox = view.$el.find('.hideDownloadCheckbox');
|
||||||
|
|
||||||
expect($hideDownloadCheckbox.length).toBeFalsy();
|
expect($hideDownloadCheckbox.length).toBeTruthy();
|
||||||
});
|
});
|
||||||
|
|
||||||
it('checkbox is checked when the setting is enabled', function () {
|
it('checkbox is checked when the setting is enabled', function () {
|
||||||
|
|
Loading…
Reference in New Issue