Skip to content

Commit d5b5399

Browse files
authored
Merge pull request #17826 from craftcms/bugfix/17824-cant-select-disabled-elements
elements with disabled class shouldn't be selectable
2 parents d2b62a8 + 5d8d244 commit d5b5399

File tree

4 files changed

+6
-2
lines changed

4 files changed

+6
-2
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
- Fixed an authorization error that could occur when editing a provisional draft of a nested element.
1313
- Fixed a bug where custom field values could be lost when changing an entry type. ([#17821](https://github.com/craftcms/cms/issues/17821))
1414
- Fixed a bug where non-admin users couldn’t edit recursively-nested elements if they didn’t have save permissions for the top-level element.
15+
- Fixed a bug where it was possible to select disabled elements within element select modals. ([#17824](https://github.com/craftcms/cms/issues/17824))
1516
- Fixed a styling issue.
1617

1718
## 5.8.16 - 2025-08-29

src/web/assets/cp/dist/cp.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/web/assets/cp/dist/cp.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/web/assets/cp/src/js/BaseElementIndexView.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,9 @@ Craft.BaseElementIndexView = Garnish.Base.extend(
233233
},
234234

235235
canSelectElement: function ($element) {
236+
if ($element.hasClass('disabled')) {
237+
return false;
238+
}
236239
if (this.settings.canSelectElement) {
237240
return this.settings.canSelectElement($element);
238241
}

0 commit comments

Comments
 (0)