meshery.io icon indicating copy to clipboard operation
meshery.io copied to clipboard

Bug Fixed

Open UtkarshMishra12 opened this issue 2 years ago • 5 comments

Signed-off-by: Utkarsh Mishra [email protected]

Description

This PR fixes #843

Notes for Reviewers

Signed commits

  • [x] Yes, I signed my commits.

UtkarshMishra12 avatar Oct 05 '22 10:10 UtkarshMishra12

Deploy Preview for mesheryio-preview ready!

Name Link
Latest commit a41e2005b6d200fbb25bb0cfcc2ac26b353b3bfa
Latest deploy log https://app.netlify.com/sites/mesheryio-preview/deploys/63495e043c25c9000b242bf0
Deploy Preview https://deploy-preview-864--mesheryio-preview.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.

netlify[bot] avatar Oct 05 '22 10:10 netlify[bot]

I'm not able to upload the video for both before the update and after the update. So may I upload it on the drive and paste the link here? @leecalcote @legendarykamal

UtkarshMishra12 avatar Oct 05 '22 10:10 UtkarshMishra12

Before Update - Video

After Update - Video

UtkarshMishra12 avatar Oct 05 '22 10:10 UtkarshMishra12

@legendarykamal Please check it now.

UtkarshMishra12 avatar Oct 05 '22 10:10 UtkarshMishra12

@legendarykamal Gentle Reminder 😊

UtkarshMishra12 avatar Oct 06 '22 17:10 UtkarshMishra12

@UtkarshMishra12 I would suggest a change to your code.

Desc: The input box width is too small according to the width of subscribe btn (at 1229px).

Current behavior at 1229 : Screenshot 2022-10-11 103127

Expected behavior: Screenshot 2022-10-11 103219

Krishna2323 avatar Oct 11 '22 05:10 Krishna2323

@Krishna2323 Now you can check the updated size of the input field. Let me know your thoughts

UtkarshMishra12 avatar Oct 11 '22 12:10 UtkarshMishra12

Now it looks good to me and is ready to be merged. @legendarykamal @leecalcote, correct me if I am wrong.

Krishna2323 avatar Oct 11 '22 12:10 Krishna2323

Could you merge the master branch to your branch it seems, and also changes from @Nikhil-Ladha, other than that it looks good. 👍

asubedy avatar Oct 13 '22 17:10 asubedy

@Nikhil-Ladha project mailing list buttons are being addressed in a separate issue and someone is working on it.

While subscribe button could be taken care of in this itself. // @UtkarshMishra12 .

kamalbuilds avatar Oct 13 '22 18:10 kamalbuilds

@Nikhil-Ladha @legendarykamal @asubedy Please review it now, the changes are made to the subscribe button.

UtkarshMishra12 avatar Oct 14 '22 13:10 UtkarshMishra12

Gentle reminder @Nikhil-Ladha 😊

UtkarshMishra12 avatar Oct 15 '22 14:10 UtkarshMishra12