-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed Break point issue of home, from 551px to 695px screen size. #657
base: main
Are you sure you want to change the base?
Fixed Break point issue of home, from 551px to 695px screen size. #657
Conversation
Hi @07-Chandra thank you for your pr. Can you remove content about the code format? |
i'm newbie to the contribution and not able to understand error- "merging is blocked" and about content about the code format . can you please elaborate more ? |
Hi, @07-Chandra We are very sorry for the trouble caused to you because our repo does not have a formatting tool. We also don’t want to include unnecessary content in a PR, as shown in the screenshot below. The automatic formatting of your editor may cause it, so can you please remove modifications like this? |
thank you, i will modify codes and will send new PR |
i think, now i have solved issues and changed content formation. |
|
||
|
||
|
||
Expand All |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @07-Chandra, can you check here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
please check now |
<div | ||
return ( | ||
<div className="w-full"> | ||
<img |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @07-Chandra, I understand you only modified w-full
, but there are still many additional changes in this file, can you remove these? Thank you.
Hi @07-Chandra , can you update the PR so that we can get it merged? Thank you. |
What does this Pull Request do ?
i Fixed Break point issue of home, from 551px to 695px screen size which was breaking earlier but now working good.
Fixes #369
##Types of change
I modified certain fixed pixel sizes to expand to full width when the screen size is smaller.