hi,

In the home page `Our clients` region is blank when no Clients is created. Must display a dummy image instead of showing blank area.

thank you

CommentFileSizeAuthor
#4 our_clients_must_show_dummy_instead_of_blank_space_while_the_clients_are_empty-2806259-4.patch19.23 KBharivenuv
#2 our_clients_must_show_dummy_instead_of_blank_space_while_the_clients_are_empty-2806259-1.patch2.45 KBharivenuv
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

harivenu_zyxware created an issue. See original summary.

harivenuv’s picture

harivenuv
Location India
CreditAttribution: harivenuv at Zyxware Technologies commented
Assigned: harivenuv » Unassigned
Status: Active » Needs review
FileSize
our_clients_must_show_dummy_instead_of_blank_space_while_the_clients_are_empty-2806259-1.patch2.45 KB

Hi,

I have created a patch to implement the feature.

Thank you,

akhilavnair’s picture

akhilavnair
Primary language Malayalam
Location Kerala
CreditAttribution: akhilavnair at Zyxware Technologies commented
Status: Needs review » Needs work

Hi,

This patch is not working for me. When applying this patch I got the following errror:

Checking patch images/clients/client.png...
error: cannot apply binary patch to 'images/clients/client.png' without full index line
error: images/clients/client.png: patch does not apply

harivenuv’s picture

harivenuv
Location India
CreditAttribution: harivenuv at Zyxware Technologies commented
Status: Needs work » Needs review
FileSize
our_clients_must_show_dummy_instead_of_blank_space_while_the_clients_are_empty-2806259-4.patch19.23 KB

Hi akhilavnair_zyxware,

Patch issue resolved. I was missing the importance of the binary part.

akhilavnair’s picture

akhilavnair
Primary language Malayalam
Location Kerala
CreditAttribution: akhilavnair at Zyxware Technologies commented
Status: Needs review » Reviewed & tested by the community

Hi,

The patch from #4 is working for me.
Thanks for the patch.

mslabba_zyxware’s picture

mslabba_zyxware CreditAttribution: mslabba_zyxware at Zyxware Technologies commented
Status: Reviewed & tested by the community » Fixed

Successfully merged the patch

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.