Skip to content

Fix element action triggers on Craft 4 element indexes#18947

Closed
timkelty wants to merge 4531 commits into
5.xfrom
4.x
Closed

Fix element action triggers on Craft 4 element indexes#18947
timkelty wants to merge 4531 commits into
5.xfrom
4.x

Conversation

@timkelty
Copy link
Copy Markdown
Contributor

@timkelty timkelty commented May 22, 2026

Summary

  • Resolve Craft 4 menu-based element actions by finding detached action triggers after appendBodyHtml() finishes.
  • Use Craft.formatInputId() for trigger IDs so action markup matches the rest of the CP.
  • Keep the fix scoped to CP action trigger initialization.

Testing

  • Verified locally in the CP entry index with Playwright in Dia.
  • Confirmed Edit entry now opens the entry editor slideout instead of reloading the index.
  • Confirmed bulk selection still disables View entry and Edit entry as expected.

brianjhanson and others added 30 commits May 13, 2026 14:02
@brianjhanson
Converting Login related javascript to web components
ba1c809 
Just the first pass, more refinement to come
@brianjhanson
Visually hidden WC styles
95dc221
@riasvdv
Add configurable compiled templates path
29ebf18
@riasvdv
Ignore missing transaction during plugin uninstall
c996a6e
@brianjhanson
Remove dangling spinner reference
5de6deb
@brianjhanson
Update ElevatedSessionManager.js to use new events
c52976f
@brianjhanson
Cleanup
b225ebb
@brianjhanson
Build
e948c3d
@brianjhanson
Update AuthManager for new form
c90ba0c
@brianjhanson
Rebuild
3e4a4d0
@brianjhanson
Merge branch '6.x' of github.com:craftcms/cms into feature/craft-logi…
f774a29 
…n-form-web-component
@brianjhanson
Merge branch 'feature/inertia-auth' of github.com:craftcms/cms into f…
b2c7e1d 
…eature/craft-login-form-web-component
@brianjhanson
Merge branch 'feature/craft-login-form-web-component' of github.com:c…
ca6aa0e 
…raftcms/cms into feature/inertia-auth
@brianjhanson
Make sure recovery codes work
739eb90
@brianjhanson
Remove old two factor asset bundle and JS
c431f9c
@brandonkelly
Release note
da02ad9 
[ci skip]
@brandonkelly
Merge pull request #18858 from craftcms/rias/cms-2078-redirect-goes-t…
58380e2 
…o-adminadminlogin-after-migrations

[6.x] Don't double-prepend the cpTrigger
@brandonkelly
Release note
7540d2f 
[ci skip]
@brandonkelly
Merge branch '6.x' into rias/cms-2080-ignore-missing-transaction-exce…
34b63c1 
…ption-during-uninstall

# Conflicts:
#	CHANGELOG.md
@brandonkelly
Merge pull request #18862 from craftcms/rias/cms-2080-ignore-missing-…
9bf824a 
…transaction-exception-during-uninstall

[6.x] Ignore missing transaction during plugin uninstall
@brandonkelly
Merge branch '6.x' into feature/configurable-compiled-templates
cfecd3a
@brandonkelly
Release note
0529e06
@brandonkelly
Merge pull request #18861 from craftcms/feature/configurable-compiled…
a66b79e 
…-templates

[6.x] Add configurable compiled templates path
@brianjhanson
Cleanup
a035e47
@brianjhanson @claude
Fix queue polling: add backoff for no-displayable-job state and guard…
f557c07 
… error retries

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@brianjhanson
Improve check for running jobs
5f32d0c
@riasvdv
Merge branch '6.x' into rias/cms-2077-6x-response-redirecturl-method-…
821f63e 
…send-not-working
@riasvdv
Changelog
9e5a854
@riasvdv
Add missing minAuthors migration
893c922
@brandonkelly
Merge branch '6.x' into brian/cms-2076-6x-infinite-queue-polling-in-c…
8eba7f4 
…ontrol-panel
riasvdv and others added 28 commits May 20, 2026 15:19
@riasvdv
Merge branch '6.x' into feature/octane-compatibility
ead7aa2
@riasvdv
Merge branch '5.x' into 6.x
d9d29e3 
# Conflicts:
#	CHANGELOG.md
#	composer.json
#	composer.lock
#	src/controllers/ElementIndexesController.php
#	src/controllers/ElementsController.php
#	src/web/assets/matrix/dist/MatrixInput.js
#	src/web/assets/matrix/dist/MatrixInput.js.map
#	src/web/twig/Extension.php
@riasvdv
Default loginPath to false
2820ac1
@riasvdv
Fix tests
c80a0b6
@riasvdv
Fix redirects when loginPath is false
f4871be
@riasvdv
Merge pull request #18922 from craftcms/feature/auth-logo
313bbb8 
[6.x] Add back the auth page powered by logo
@brianjhanson
Make sure money value is truthy
30b5b35
@brianjhanson
Update changelog entry
c8be82b
@brandonkelly
Merge pull request #18921 from craftcms/feature/octane-compatibility
38a4a86 
[6.x] Octane compatibility fixes
@brianjhanson
Fixed an issue preventing saving when link fields were present
394ca92
@brianjhanson
Whoops, put item on correct changelog
e3737e7
@brandonkelly
Merge branch '5.x' of https://github.com/craftcms/cms into 6.x
9555ef3 
# Conflicts:
#	CHANGELOG.md
#	src/config/app.php
#	src/services/Entries.php
@brandonkelly
Merge pull request #18934 from craftcms/bugfix/save-link-fields
58473b9 
[6.x] Fixed an issue preventing saving when link fields were present
@brandonkelly
Merge branch '6.x' of https://github.com/craftcms/cms into 6.x
5c1349f
@brandonkelly
Changelog tweaks
41507e0 
[ci skip]
@brandonkelly
Merge branch '6.x' into bugfix/money-value-prep
0dec920 
# Conflicts:
#	CHANGELOG.md
@brandonkelly
Merge pull request #18930 from craftcms/bugfix/money-value-prep
be542c2 
[6.x] Make sure money value is truthy
@riasvdv
Fix invalid query filters
90df382
@riasvdv
Changelog
b5e73ad
@vnali
URL should be Url
35e28d4
@riasvdv
Merge pull request #18940 from vnali/patch-24
84b0f60 
Update changelog: URL should be Url
@riasvdv
Fix validatable preparation lifecycle
bcdf78c
@riasvdv
Changelog
3a9b9a2
@brandonkelly
Merge pull request #18937 from craftcms/rias/cms-2111-6x-query-for-no…
b8a709e 
…n-existing-section-returns-all-entries

[6.x] Fix invalid query filters
@riasvdv
Move setAttributes into trait
a527c97
@riasvdv
Rename ComponentRules to ValidatableRules
42f370a
@riasvdv
Merge branch '6.x' into feature/consistent-validatable-preparation
c421a17
@riasvdv
Fix legacy implementation
9e962b1
@timkelty timkelty closed this May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@timkelty @brianjhanson @riasvdv @brandonkelly @patarok @vnali