Hopiu/wagtail
1
0
Fork 0
mirror of https://github.com/Hopiu/wagtail.git synced 2026-03-24 01:40:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Show / hide InlinePanel arrow buttons correctly on non-default tabs

Browse source 
Base the show/hide logic on the 'deleted' class, rather than visibility, because elements on non-default tabs are considered invisible. Fixes #3667
This commit is contained in:
Matt Westcott 2017-06-28 16:11:38 +01:00 committed by Bertrand Bordage
parent 493b62b5cd
commit 29360a4f70
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
wagtail/wagtailadmin/static_src/wagtailadmin/js/page-editor.js
View file

@ -187,7 +187,7 @@ function InlinePanel(opts) {
self.updateMoveButtonDisabledStates = function() {
if (opts.canOrder) {
var forms = self.formsUl.children('li:visible');
var forms = self.formsUl.children('li:not(.deleted)');
forms.each(function(i) {
$('ul.controls .inline-child-move-up', this).toggleClass('disabled', i === 0).toggleClass('enabled', i !== 0);
$('ul.controls .inline-child-move-down', this).toggleClass('disabled', i === forms.length - 1).toggleClass('enabled', i != forms.length - 1);
@ -210,7 +210,7 @@ function InlinePanel(opts) {
self.animateSwap = function(item1, item2) {
var parent = self.formsUl;
var children = parent.children('li:visible');
var children = parent.children('li:not(.deleted)');
// Apply moving class to container (ul.multiple) so it can assist absolute positioning of it's children
// Also set it's relatively calculated height to be an absolute one, to prevent the container collapsing while its children go absolute