-
Notifications
You must be signed in to change notification settings - Fork 79
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
add text and tooltip for copy success msg #16
Conversation
ad50357
to
3a6b366
Compare
hey @nehagup and @Hermione2408 review my changes |
Hey @sk-pathan |
hey @Hermione2408 , |
d36e5b9
to
76e44fb
Compare
Hey @sk-pathan I noticed the second gif is overlapping in the medium size view. Also, there's inconsistency with the text color across different views; please make it consistent to text-secondary-300 |
hi @Hermione2408 have look on change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @sk-pathan In medium size, now gif is not aligned with the side div
components/features.tsx
Outdated
@@ -94,9 +107,23 @@ const CopyCode = () => { | |||
|
|||
return ( | |||
<> | |||
<p className='font-bold text-secondry-300'>Try Keploy Locally </p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
secondary*
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sk-pathan please correct this spelling mistake the class is not getting applied
2b3258d
to
8954b78
Compare
Signed-off-by: sk-pathan <[email protected]>
Signed-off-by: sk-pathan <[email protected]>
Signed-off-by: sk-pathan <[email protected]>
Signed-off-by: sk-pathan <[email protected]>
8954b78
to
443f46d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
components/features.tsx
Outdated
@@ -94,9 +107,23 @@ const CopyCode = () => { | |||
|
|||
return ( | |||
<> | |||
<p className='font-bold text-secondry-300'>Try Keploy Locally </p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sk-pathan please correct this spelling mistake the class is not getting applied
Signed-off-by: sk-pathan <[email protected]>
8a19d76
to
f384f5e
Compare
Hey @sk-pathan, this issue is now closed keploy/keploy#1672. Thanks for your contributions I am closing this PR! |
fix:#1672
what pr do
This pull request aims to enhance the user experience and streamline the development process by introducing two key features: "Try Deploy Locally" button and "Copy Tooltip for Success Message."
Changes Proposed:
Added "Try Deploy Locally" button to enable users to test deployments locally before committing changes.
Implemented a copy tooltip for success messages, providing users with clear feedback upon successful copy actions.
mobile view:
https://github.com/keploy/website/assets/141400463/c4f462ea-383a-4dc6-a47b-6adb55d56230
laptop view:
laptop1672.mp4