- Issue created by @Shubham_Kumar
- @shubham_kumar opened merge request.
- Status changed to Needs review
4 months ago 5:48am 10 September 2024 Hello,
I came across the same problem of missing screenshot in Appearance section. I've installed the theme version 1.0.7 in Drupal version 10.3.6.
When I visited/admin/appearanceto install the theme, I saw the missing screenshot. I applied the MR!15, which resolved the issue. I've reviewed the MR and it seems to be a typo error in the naming of the image file, which was addressed.
IMO we should fix this.
Attached ss of before and after patch. Hence, moving to RTBC.
Thanks- Issue was unassigned.
- Status changed to RTBC
about 2 months ago 7:24am 23 November 2024 - First commit to issue fork.
- 🇮🇳India akulsaxena
Rebased.
The missing trailing comma's were causing PHPCS pipeline to fail so added trailing comma's in the twig file
MR seems to be working fine, Ready to merge now. - 🇮🇳India yogesh.k Gurgaon
yogesh.k → made their first commit to this issue’s fork.
-
yogesh.k →
committed 428b9355 on 1.0.x authored by
shubham_kumar →
Issue #3473227: Screenshot is not visible
-
yogesh.k →
committed 428b9355 on 1.0.x authored by
shubham_kumar →