diff --git a/docs/gotcha.txt b/docs/gotcha.txt index 6151f22b5..6198ae206 100644 --- a/docs/gotcha.txt +++ b/docs/gotcha.txt @@ -12,17 +12,6 @@ save you many hours of grief. -------------------------------------------------------------------- * WebGUI now requires XML::FeedPP version 0.40 or greater. - * Due to a change in the way that assets are dragged in Page Layouts, - custom layout templates need to be changed to add another div: - - before: -
- after: -
- - All default WebGUI Page Layout templates have already been updated. - - 7.7.2 -------------------------------------------------------------------- * WebGUI now requires Clone version 0.31 or greater. diff --git a/docs/upgrades/packages-7.7.4/root_import_layout.wgpkg b/docs/upgrades/packages-7.7.4/root_import_layout.wgpkg deleted file mode 100644 index 428695fab..000000000 Binary files a/docs/upgrades/packages-7.7.4/root_import_layout.wgpkg and /dev/null differ diff --git a/lib/WebGUI/Asset/Wobject/Layout.pm b/lib/WebGUI/Asset/Wobject/Layout.pm index 5965308ed..e854ad2c4 100644 --- a/lib/WebGUI/Asset/Wobject/Layout.pm +++ b/lib/WebGUI/Asset/Wobject/Layout.pm @@ -175,6 +175,7 @@ sub prepareView { } my %vars; + $vars{showAdmin} = ($session->var->isAdminOn && $self->canEdit && $self->canEditIfLocked); my $splitter = $self->{_viewSplitter} = $self->getSeparator; @@ -194,9 +195,12 @@ sub prepareView { $child->prepareView; $placeHolder{$assetId} = $child; push @children, { - id => $assetId, - isUncommitted => $child->get('status') eq 'pending', - content => $splitter . $assetId . '~~', + id => $assetId, + isUncommitted => $child->get('status') eq 'pending', + content => $splitter . $assetId . '~~', + }; + if ($vars{showAdmin}) { + $children[-1]->{'dragger.icon'} = sprintf '
%s
', $assetId, $session->icon->drag('class="dragTrigger"'); }; } @@ -230,7 +234,6 @@ sub prepareView { unshift @{ $vars{"position1_loop"} }, reverse @children; } - $vars{showAdmin} = ($session->var->isAdminOn && $self->canEdit && $self->canEditIfLocked); if ($vars{showAdmin}) { # under normal circumstances we don't put HTML stuff in our code, but this will make it much easier # for end users to work with our templates @@ -247,7 +250,6 @@ sub prepareView { } '); - $vars{"dragger.icon"} = '
'.$session->icon->drag('class="dragTrigger"').'
'; $vars{"dragger.init"} = '