- Issue created by @shweta__sharma
- First commit to issue fork.
- @priya_degwekar opened merge request.
- Status changed to Needs work
over 1 year ago 5:40am 6 June 2023 Hi @Priya_Degwekar,
I have verified your MR#3 on 9.5.8, the issue is fixed, but it is not fixed on the screen between 777px to 769px.
Also, you have to put CSS code first into the sass file then it will compile and add directly into the style.css file because in this project sass is used.- @priya_degwekar opened merge request.
- Assigned to Priya_Degwekar
- 🇮🇳India Priya_Degwekar
Hi @shweta__sharma , Please review MR#5. I have added some media queries in _header.scss file. Please compile it then check. Issue is fixed at my end. Thanks!
- Status changed to Needs review
over 1 year ago 12:52pm 8 June 2023 - Issue was unassigned.
- Assigned to Jay Jangid
- Issue was unassigned.
- 🇮🇳India Jay Jangid
Hey,@Priya_Degwekar Thanks for creating MR,
but after applying this MR it's showing warning: 2 lines add whitespace errors.
See screenshot.
Thank you. - @priya_degwekar opened merge request.
- 🇮🇳India Priya_Degwekar
Hi, I have worked on fixing whitespace warnings of #MR5. Now This below patch looks clean. Attaching the patch here.
Please review.
Thanks! - 🇮🇳India riddhi.addweb
I have applied the MR !6 patch, it solves the issue.
But the issue still exists for the Resolution between "601px to 762px".
Please check the attached screenshots for the reference. - Status changed to Needs work
5 months ago 8:45am 19 June 2024 Hi, The MR!6 looks good on other resolution but it need to be fixed on the screen around 760px to 601px.
Attached screenshot for reference.
So changing the status to Needs work.
Thanks.