Skip to content

Conversation

@DhairyaMajmudar
Copy link

@DhairyaMajmudar DhairyaMajmudar commented Jan 11, 2024

Fix :-

  1. 🐞 Bug: Issues with the sidebar in forum section.  CircuitVerse#4554
  2. 🐞 Bug: Overflow in medium and small screens.  CircuitVerse#4575
  • Fixing Responsiveness issues in the medium-sized screen ( Tablets / large mobiles )

Before ScreenShot
image

After ScreenShot
image

@DhairyaMajmudar
Copy link
Author

@tanmoysrt pls. review.

Copy link
Member

@tanmoysrt tanmoysrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of squeezing the sidebare you can try to squeeze other contents.

@DhairyaMajmudar
Copy link
Author

Instead of squeezing the sidebare you can try to squeeze other contents.

By internal content do you mean internal divs?

@tanmoysrt
Copy link
Member

Instead of squeezing the sidebare you can try to squeeze other contents.

By internal content do you mean internal divs?> Instead of squeezing the sidebare you can try to squeeze other contents.

I mean for tab like size , in sidebar try to keep the buttons same as desktop, the text in single line rather than breaking in two line

@DhairyaMajmudar DhairyaMajmudar changed the title Fixing responsiveness in medium screens. Fixing responsiveness and overlfow issues Jan 12, 2024
@DhairyaMajmudar
Copy link
Author

@tanmoysrt I have updated my pr pls. take a look

@DhairyaMajmudar DhairyaMajmudar changed the title Fixing responsiveness and overlfow issues Fix: Fixing responsiveness and overlfow issues Feb 15, 2024
@DhairyaMajmudar
Copy link
Author

DhairyaMajmudar commented Feb 24, 2024

@tanmoysrt I have made the suggested changes, if it looks good can you pls. merge the pr?

cc: @vedant-jain03

@ThatDeparted2061
Copy link
Member

@DhairyaMajmudar Would you be open to continue this PR ?
You would have to resolve the conflicts and upload another screenshot of the changes !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants