- Issue created by @nginex
- last update
over 1 year ago Composer require failure - @nginex opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 4:49pm 15 May 2023 - π©πͺGermany Anybody Porta Westfalica
Yeah good point @nginex! The thumbnail field on media has never been a good thing (from my perspective), but yes, we should support it, if possible.
Still Photoswipe is for real image fields and the thumbnail is not a "real" image field. So this isn't a bug, but a nice to have feature request I agree with! :)
- Status changed to Needs work
over 1 year ago 8:56am 30 May 2023 - π©πͺGermany Anybody Porta Westfalica
Please add a test for this case to show it didn't work before and works now.
Also please test that the Thumbnail option doesn't appear in cases where it should not. - Open on Drupal.org βCore: 10.0.7 + Environment: PHP 8.2 & MySQL 8last update
over 1 year ago Waiting for branch to pass - Assigned to nginex
- π©πͺGermany Anybody Porta Westfalica
3.x is now 4.x
@nginex any plans for further work on this? (Adding tests)
- π©πͺGermany Anybody Porta Westfalica
@Grevil: Could you please check if the same issue still exists in 5.x? That should be the focus.
- πΊπ¦Ukraine nginex
@Anybody,
I've planned it for this week, thanks for reminding.
- π©πͺGermany Anybody Porta Westfalica
New features go into 5.x any further plans @nginex?