Background colour of the content region is different as one can see in the screenshot of the theme.

CommentFileSizeAuthor
#6 Screen Shot 2020-03-25 at 7.36.06 PM.png15.87 KBdjsagar
#2 background-color-of-region-content-3107976-1.patch388 bytesBhumikaVarshney
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

BhumikaVarshney created an issue. See original summary.

BhumikaVarshney’s picture

BhumikaVarshney CreditAttribution: BhumikaVarshney as a volunteer and at OpenSense Labs commented
FileSize
background-color-of-region-content-3107976-1.patch388 bytes

I have added a patch for this issue , Please review it and let me know if it works

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor commented
Status: Active » Needs work
BhumikaVarshney’s picture

BhumikaVarshney CreditAttribution: BhumikaVarshney as a volunteer and at OpenSense Labs commented
Status: Needs work » Needs review
djsagar’s picture

djsagar CreditAttribution: djsagar at OpenSense Labs commented
Status: Needs review » Needs work

Hi @BhumikaVarshney,

Patch is not applied, I find error.

error: patch failed: css/layout.css:252
error: css/layout.css: patch does not apply

Thanks!

djsagar’s picture

djsagar CreditAttribution: djsagar at OpenSense Labs commented
FileSize
Screen Shot 2020-03-25 at 7.36.06 PM.png15.87 KB

Here the attachment for error.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor commented
Status: Needs work » Reviewed & tested by the community

Changes pushed by applying the suggestion from the patch manually and fixed all the coding standards as well.

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant commented

gaurav.kapoor’s picture

gaurav.kapoor CreditAttribution: gaurav.kapoor at Axelerant commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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