Skip to content

Menu-bar width & Responsiveness fixed#103

Open
faizalmsdev wants to merge 2 commits intoPritamSarbajna:mainfrom
faizalmsdev:navbar-width-fixed
Open

Menu-bar width & Responsiveness fixed#103
faizalmsdev wants to merge 2 commits intoPritamSarbajna:mainfrom
faizalmsdev:navbar-width-fixed

Conversation

@faizalmsdev
Copy link
Copy Markdown

@faizalmsdev faizalmsdev commented Jun 12, 2023

As per your advice i just fixed the width of the menu-bar and its become more responsiveness.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 12, 2023

Deploy Preview for simple-tourism-organization ready!

Name Link
🔨 Latest commit 9e27ce9
🔍 Latest deploy log https://app.netlify.com/sites/simple-tourism-organization/deploys/6486f9bbe8883c0007617abe
😎 Deploy Preview https://deploy-preview-103--simple-tourism-organization.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Copy Markdown
Owner

@PritamSarbajna PritamSarbajna left a comment

Choose a reason for hiding this comment

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

The PR needs following changes :

  • The menu bar should cover the whole area or 90% from the middle.
  • When for closing I'm pressing the menu it is shrinking and then closing. Please fix it.
  • Menu Bar design is good.

@faizalmsdev
Copy link
Copy Markdown
Author

Noted , you mean when i click the menu bar its need to open onfull width of home screen?

@PritamSarbajna
Copy link
Copy Markdown
Owner

Noted , you mean when i click the menu bar its need to open onfull width of home screen?

Yeah if not full then atleast 90%.

Copy link
Copy Markdown

@codeCraft-Ritik codeCraft-Ritik left a comment

Choose a reason for hiding this comment

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

Nice approach — the logic flows well and is readable.

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