- Issue created by @Nitin shrivastava
- 🇮🇳India Nitin shrivastava
The screen shot and the theme description have been updated in the patch please test.
- Status changed to Needs review
10 months ago 5:51am 13 February 2024 - Status changed to Needs work
10 months ago 6:36am 13 February 2024 @Nitin shrivastava , verified #2 patch and the screenshot of the theme showing broken.
- Assigned to sourojeetpaul
Yes, the screenshot seems to be broken actually.
Working on it and raising an MR for the same!- Issue was unassigned.
- Status changed to Needs review
9 months ago 9:42am 3 April 2024 Hi, I've updated the info.yml by updating the screenshot reference over there and raised an MR for the same. Please have a look:
https://git.drupalcode.org/project/animal_shelter/-/merge_requests/15
The screenshot depicts how it looks on the homepage when the theme is set as default theme. Also in my instance I've implemented a caraousel as a block and placed it on the banner region that's why it looks like this, you can customize it according to your choice as well. I thought it might give a good reference to the visual appearance of the theme. This is how it looks on the appearance page now:
- 🇮🇳India Yashaswi18
Hello, I applied the MR!15 and verified, I can now see the image for the theme in the appearance tab. Attaching the screenshot.
- Status changed to RTBC
7 months ago 4:38am 28 May 2024 - 🇮🇳India Vivek Panicker Kolkata
As the issue has already been tested by @Yashaswi18 and things are fine, updating the issue state to RTBC.
- Status changed to Needs work
about 1 month ago 6:11am 7 November 2024 I think the screenshot can be improved further to showcase the functionality in a more vivid and attractive way! We can go about with the screenshot which is being used on the project page itself.
I'll ask my design team to come up with a nice screenshot for this one before shipping in! IMO stitching the banner and services section into a single image which will confirm to Drupal Theme screenshots dimensions and maintaining a reasonable file size will be a good point to start off!
I was working on it initially when I had very little design knowledge, and now after working on a plethora of PB logo issues, this seems a bit off, hence moving it back to NW :)- 🇮🇳India manojsaha Kolkata
I've made the suggested changes. Please a have a look..
Hi @manojsaha,
Thanks for promptly acting on it! The changes look good and ready to ship in. I'm updating the MR :)-
sourojeetpaul →
committed b2d9c9e4 on 1.0.x
Issue #3421034: Need to update the theme screenshot for better visuals
-
sourojeetpaul →
committed b2d9c9e4 on 1.0.x
Thank you all for your active participation on the issue. It'll be added on the new release :)
Automatically closed - issue fixed for 2 weeks with no activity.