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.

Sign up for GitHub

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

Jump to bottom

Fix Spacing Issue between Subscribe button and input in CTA #4757

Closed
arpitghura opened this issue Aug 18, 2023 · 12 comments · Fixed by #4825
Closed

Fix Spacing Issue between Subscribe button and input in CTA #4757

arpitghura opened this issue Aug 18, 2023 · 12 comments · Fixed by #4825
Assignees
@arpitghura
Labels
kind/enhancement New feature or request

Comments

@arpitghura
Copy link
Contributor

Current Behavior

image

Desired Behavior

Screenshots / Mockups

Implementation

Acceptance Tests

Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Join the Layer5 Community by submitting your community member form.
@arpitghura arpitghura added the kind/enhancement New feature or request label Aug 18, 2023
@arpitghura
Copy link
Contributor Author

I would like to work on this issue.

@VivekBaldha0712
Copy link
Member

i would like to work on this.

@Yagi91
Copy link

Yagi91 commented Aug 20, 2023

Please assign this to me

@iArchitSharma
Copy link
Contributor

@arpitghura, could you please provide further explanation regarding this issue? Specifically, could you indicate the screen size at which this issue occurs and provide the link to the page where it can be observed?

@arpitghura
Copy link
Contributor Author

@arpitghura, could you please provide further explanation regarding this issue? Specifically, could you indicate the screen size at which this issue occurs and provide the link to the page where it can be observed?

The spacing issue is there till max-width: 768px. This issue is seen on the website's main page https://layer5.io/.

@iArchitSharma
Copy link
Contributor

iArchitSharma commented Aug 26, 2023 •
edited
Loading

@arpitghura, I checked, and the issue only appears for a width size of 768px and 769px. I believe the desired behavior of this section or CTA should be the same as at 767px.

@Abiey2579
Copy link
Contributor

@arpitghura have you resolved the issue yet? Or still on it?

@arpitghura
Copy link
Contributor Author

@arpitghura have you resolved the issue yet? Or still on it?

I am waiting for the assignment of task. As soon as task is assigned, I will start working on it.

@iArchitSharma
Copy link
Contributor

@arpitghura, maybe you should discuss it on Slack or during the next development meeting.

@vishalvivekm
Copy link
Contributor

@arpitghura

Let's discuss this on Today's websites call. Please Join the meeting : https://meet.layer5.io/websites at 5:30 PM IST.
Meeting minutes: https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit
// @iArchitSharma @Abiey2579

@arpitghura
Copy link
Contributor Author

@vishalvivekm I have discussed this issue in the meet. Can you please assign it to me.

@abhijeetgauravm
Copy link
Contributor

Here u go @arpitghura

@arpitghura arpitghura mentioned this issue Sep 5, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arpitghura arpitghura

Labels
kind/enhancement New feature or request
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes Spacing and Margin Issue in the Subscribe Section. arpitghura/layer5
7 participants
@arpitghura @Abiey2579 @abhijeetgauravm @iArchitSharma @Yagi91 @VivekBaldha0712 @vishalvivekm