Skip to content

Commit

Permalink
Merge pull request #15717 from craftcms/bugfix/15712-remove-card-chip…
Browse files Browse the repository at this point in the history
…-layout-shift

Bugfix/15712 remove card chip layout shift
  • Loading branch information
brandonkelly authored Sep 12, 2024
2 parents 00111c2 + b1ab63f commit d3b1fa4
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
- Fixed an error that could occur when saving an entry with a Matrix field, if the nested entries didn’t have slugs.
- Fixed a bug where relation fields weren’t merging uploaded asset IDs with the existing field values. ([#15707](https://github.com/craftcms/cms/issues/15707))
- Fixed a styling issue with inline-editable Matrix block tabs. ([#15703](https://github.com/craftcms/cms/issues/15703))
- Fixed a bug where the control panel layout could shift briefly when removing an element from an element select input. ([#15712](https://github.com/craftcms/cms/issues/15712))
- Fixed an RCE vulnerability.
- Fixed an XSS vulnerability.

Expand Down
2 changes: 1 addition & 1 deletion src/web/assets/cp/dist/cp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/web/assets/cp/dist/cp.js.map

Large diffs are not rendered by default.

3 changes: 3 additions & 0 deletions src/web/assets/cp/src/js/BaseElementSelectInput.js
Original file line number Diff line number Diff line change
Expand Up @@ -642,11 +642,14 @@ Craft.BaseElementSelectInput = Garnish.Base.extend(

animateElementAway: function ($element, callback) {
const offset = $element.offset();
const width = $element.width();

$element.appendTo(Garnish.$bod).css({
'z-index': 0,
position: 'absolute',
top: offset.top,
left: offset.left,
maxWidth: width + 'px',
});

const animateCss = {
Expand Down

0 comments on commit d3b1fa4

Please sign in to comment.