- First commit to issue fork.
- Status changed to Needs review
over 1 year ago 9:53pm 28 December 2023 - heddn Nicaragua
Tests are mostly passing now on the MR. At least the same tests that aren't failing on HEAD are now passing. Hiding patches.
- First commit to issue fork.
- Status changed to RTBC
12 months ago 12:32pm 3 September 2024 - π§π¬Bulgaria pfrenssen Sofia
This is looking good to me. I tried it out and it works brilliantly. The context data provided is very complete and helpful. One gotcha was that the Views display is not retained, so switching to another display cannot be done in the normal way (by calling
ViewsExecutable::setDisplay('my_display')
).This is not a problem in practice since the widget configuration is also passed by reference in the alter hook, so it can be done by setting
$configuration['view_display'] = 'my_display'
.
- First commit to issue fork.
- π©πͺGermany Anybody Porta Westfalica
Some very clever people involved here, all comments resolved. So let's merge this in finally.
-
anybody β
committed 2ec25804 on 8.x-2.x authored by
heddn β
Issue #3038854 by heddn, edysmp, pfrenssen, oknate, anybody, geek-merlin...
-
anybody β
committed 2ec25804 on 8.x-2.x authored by
heddn β