- Issue created by @sanskar mishra
- First commit to issue fork.
- Merge request !8animal_shelter-3417690:Added box-shadow to the header. → (Open) created by shweta__sharma
- Status changed to Needs review
11 months ago 10:08am 7 February 2024 - Status changed to Needs work
about 2 months ago 12:36pm 6 November 2024 Hello,
Thanks for reporting the issue and working on it. There was legit no clear distinction between menu and main content. After applying the patch that issue is resolved, the MR also looks good.
On the front page it looks absolutely fine to me, but on the individual node page, I feel a scarcity of breathing space between menu and the breadcrumb. Attaching both the screenshots for quick refs. Please have a look and will be highly appreciated if you come up with any solution to fix that. Hence I'm moving that back to NW :(Thanks and Regards!
- First commit to issue fork.
- 🇮🇳India jaydeep_patel Ahmedabad
Hello @sourojeetpaul,
I've updated the MR!8 and added padding on top and bottom of the Breadcrumb as per your comment.
Thank you
- 🇮🇳India jaydeep_patel Ahmedabad
Hello @sourojeetpaul,
I've updated the MR!8 and added padding on top and bottom of the Breadcrumb as per your comment.
Thank you
- 🇮🇳India akulsaxena
sourojeetpaul
I resolved the merge conflicts as asked,
Can you please give it a look - 🇮🇳India jaydeep_patel Ahmedabad
MR !8 applied successfully without any conflict and looks good now.
Thanks @akulsaxena for resolve conflict.Moving this issue to RTBC.