Skip to content
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

fix: banner button fixed #2954

Merged
merged 3 commits into from
May 22, 2024

Conversation

Aniket979Kanodia
Copy link
Contributor

Description

  • Modified parent div of banner to increase size if size of inside content increases

Related issue(s)

Resolves #2953

Copy link

netlify bot commented May 11, 2024

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 5361180
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/664d93f0b04d35000889cc82
😎 Deploy Preview https://deploy-preview-2954--asyncapi-website.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 configuration.

@Aniket979Kanodia Aniket979Kanodia changed the title Banner button fixed fix: Banner button fixed May 11, 2024
@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented May 11, 2024

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 29
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🟠 PWA 56

Lighthouse ran on https://deploy-preview-2954--asyncapi-website.netlify.app/

@Aniket979Kanodia Aniket979Kanodia changed the title fix: Banner button fixed fix: banner button fixed May 11, 2024
Copy link
Member

@akshatnema akshatnema left a comment

Choose a reason for hiding this comment

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

image

@Aniket979Kanodia Can we expand this Banner more in mobile view, without hindering any web view?

Copy link
Member

@Mayaleeeee Mayaleeeee left a comment

Choose a reason for hiding this comment

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

I think it's okay @akshatnema
He just need to position it in the middle of the frame.

Screenshots_2024-05-21-07-34-35.png

@akshatnema
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit 6b0f230 into asyncapi:master May 22, 2024
23 checks passed
@akshatnema
Copy link
Member

@Aniket979Kanodia can you please replicate these changes in migrate-ts branch as well?

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

Successfully merging this pull request may close these issues.

[BUG] "Apply To Speak" button on landing page is partially visible in mobile view
4 participants