Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Steps to reproduce:
Install standard profile
Add a new node
Click the new "Edit" toolbar link
Edit a field, save, click anywhere to un-grey-out the editing interface
Click the "Menu" toolbar link
Expectation:
The menu drawer which was collapsed should show
Result:
The overlay opens the /admin page
Comments
Comment #1
Wim LeersIncredible that none of us found this during testing :(
Pinging Jesse Beach, since she's most knowledgeable on Toolbar.
Comment #2
jessebeach CreditAttribution: jessebeach commentedI'm pretty sure this will end up being this bug:
#1858576: The $toolbar variable in toolbar.js is set to an empty jQuery object after an AJAX request
Comment #3
energee CreditAttribution: energee commentedI noticed this on the D7 version. See #1852164: Navbar offset not being set correctly, will roll a D8 patch. (If its the same)
Comment #4
tim.plunkettExpanding the original steps:
Close the overlay.
Click Edit.
Expected:
Edit grey-overlay thing with selectable fields
Result:
Edit grey-overlay thing, no fields to select, and:
"Uncaught TypeError: Cannot read property 'document' of null "
Comment #5
David_Rothstein CreditAttribution: David_Rothstein commentedYeah, I've seen this on other things too; it's not related to Edit module.
A simple way to reproduce it is to go to admin/structure/views/add and change the view from Content to Users - that kicks off an Ajax request which triggers this.
The patch I just posted at #1858576: The $toolbar variable in toolbar.js is set to an empty jQuery object after an AJAX request seems to fix both cases of it.
Comment #6
Wim LeersI'm glad to hear that Edit didn't break other things and that a solution is available :)