Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Contextual link to edit queue is not showing with views that have relation and sort on any entityqueue.
Patch will follow
Comment | File | Size | Author |
---|---|---|---|
#23 | 2825773-23-outside_in_to_settings_tray.patch | 960 bytes | SvenVdv |
#20 | 2825773-20-followup.patch | 1.03 KB | amateescu |
#18 | 2825773-18-followup.patch | 431 bytes | amateescu |
#14 | interdiff.txt | 2.18 KB | amateescu |
#14 | 2825773-13.patch | 5.42 KB | amateescu |
Comments
Comment #2
josebc CreditAttribution: josebc at Vardot commentedComment #3
Mohammed J. RazemComment #4
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commented+1 Testing :)
Comment #5
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #6
josebc CreditAttribution: josebc at Vardot commentedAdded "Settings Tray" support if enabled
Comment #7
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commented+1 Having more testing.
Comment #8
amateescu CreditAttribution: amateescu as a volunteer commentedNice! The patch does make the contextual link appear for views, but the outside-in part doesn't seem to work.. should we remove it from the patch?
Comment #9
josebc CreditAttribution: josebc at Vardot commentedWe have it working using this patch, what theme are you using?
Comment #10
amateescu CreditAttribution: amateescu as a volunteer commentedI'm testing on a default D8 install, so using the bartik theme.
Comment #11
josebc CreditAttribution: josebc at Vardot commentedThats odd, i just tested on with both 8.2 and 8.3, are there any js errors in console?
Comment #12
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedI'm in with more testing ...... and automating the test.
I feel that the logic in the code is right,
Comment #14
amateescu CreditAttribution: amateescu as a volunteer commentedSpent some more time digging into this and I found out that the reason why I wasn't getting the right behavior with the settings tray is because contextual links are cached in the browser's session storage, and that's not cleared by the regular Drupal cache clearing mechanism.
Anyway, it does work nicely indeed, great work! I cleaned up the patch a little (see attached interdiff) and committed to 8.x-1.x. Thanks!
Comment #15
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedA preview of Drupal 8 "Outside-In" with Entity Queues.
This is done utilizing Sportsleague distribution and Entityqueue module with the Contextual Links patch.
https://www.youtube.com/watch?v=vr-AqQmLoGs
Comment #17
SvenVdv CreditAttribution: SvenVdv commentedI get the "edit subqueue" option in the menu, but it doesn't open in a settings tray, it opens in a new page. I think it has to do with the fact that the module is now called Settings Tray (instead of Outside-in)?
Comment #18
amateescu CreditAttribution: amateescu as a volunteer commentedThat's a very good point. Can you try this patch?
Comment #19
SvenVdv CreditAttribution: SvenVdv commentedDoes the "data-ouside-in-edit" also have to be "data-settings-tray-edit" etc. ?
Comment #20
amateescu CreditAttribution: amateescu as a volunteer commentedIndeed, and there's another instance that we need to update as well at the top of the function.
Comment #21
SvenVdv CreditAttribution: SvenVdv commentedI had this one (off_canvas), but it still doesn't work.
Comment #22
SvenVdv CreditAttribution: SvenVdv commentedSorry, wrong file :)
Comment #23
SvenVdv CreditAttribution: SvenVdv commentedI see I forgot to remove the "in" -_- Still doesn't work though.
Comment #24
SvenVdv CreditAttribution: SvenVdv commentedIt seems that the display for the entityqueue subqueue items field template has to be set to "default" instead of "minimal" ^^ It works now.
Comment #26
amateescu CreditAttribution: amateescu as a volunteer commentedNice, committed and pushed the followup patch to 8.x-1.x.
Comment #27
SvenVdv CreditAttribution: SvenVdv at Calibrate commentedComment #28
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThank you SvenVdv, Andrei I will use the patch until we have a new release for the entityqueue module.
Your work saves time.
Thanks for having this issue fixed after the Drupal core team had outside_in renamed to settings_tray in Drupal ^8.4.0 the folder and code!!
Having more testing ....
Comment #30
jwilson3I have a view configured with a block display and a required entity queue relationship. I have settings tray module enabled. However I neither see the contextual links on the block wrapper, nor in the view wrapper. If I enter into the edit mode and edit the block. there are no options in the sidebar to edit the items in the entity queue.
What am I doing wrong here?
Comment #31
amateescu CreditAttribution: amateescu as a volunteer commented@jwilson3, it seems that we only show the contextual links when the view uses the entityqueue sort handler, but I don't see any good reason for that limitation so I wrote a patch for it here: #2957971: Show the 'Edit subqueue' contextual link even when the view does not use the entityqueue sort handler.
Can you test it and let me know if it works for you?