Problem/Motivation

No screenshot is present in the theme installation which is not looking good.

Example

CommentFileSizeAuthor
#10 Screenshot from 2024-02-14 17-40-06.png206.27 KBkaranpagare
#7 patch_2_issue.png24.91 KBtanuja bohra
#7 image_size_issue.png206.13 KBtanuja bohra
#2 3421039-1.patch488 bytesnitin shrivastava
Screenshot 2024-02-13 113511.png17.26 KBnitin shrivastava

Issue fork aerodash-3421039

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Nitin shrivastava created an issue. See original summary.

nitin shrivastava’s picture

nitin shrivastava commented
StatusFileSize
new 3421039-1.patch488 bytes

Added theme image and created patch.

Raveen Thakur made their first commit to this issue’s fork.

Raveen Thakur opened merge request !2

raveen_thakur51’s picture

raveen_thakur51 commented

.

raveen_thakur51’s picture

raveen_thakur51 commented
Status: Needs work » Needs review
tanuja bohra’s picture

tanuja bohra commented
StatusFileSize
new image_size_issue.png206.13 KB
new patch_2_issue.png24.91 KB

@Nitin shrivastava, @Raveen Thakur - The #2 patch shows a broken theme screenshot and MR2 is working but the theme screenshot is not covering the section properly like other themes.

tanuja bohra’s picture

tanuja bohra commented
Status: Needs review » Needs work
raveen_thakur51’s picture

raveen_thakur51 commented
Status: Needs work » Needs review
karanpagare’s picture

karanpagare commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2024-02-14 17-40-06.png206.27 KB

Tested MR !2 , screenshot looks better now and fills screen.
Making RTBC for now.

yogesh.k made their first commit to this issue’s fork.

yogesh.k’s picture

yogesh.k commented
Status: Reviewed & tested by the community » Fixed
yogesh.k’s picture

yogesh.k commented
Status: Fixed » Closed (fixed)