-
Notifications
You must be signed in to change notification settings - Fork 90
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
feat: add orbiting circles for community section on landing page #88
Conversation
Signed-off-by: techmannih <[email protected]>
You just copy pasted exact same changes I did. |
@khareyash05 please review |
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.
Could you please address the above comment. Thanks
Signed-off-by: techmannih <[email protected]>
please check once it's totally changed, @khareyash05 also check please |
Closing due to unaddressed comments |
@khareyash05 Which comment did I not reply to? I am working on the issue. You can also check that this is not copied code. |
components/community.tsx
Outdated
@@ -59,7 +63,7 @@ function SocialLinkCard({ | |||
rel="noopener noreferrer" | |||
style={showExtraStyle ? { ...style } : {}} | |||
className={`${ | |||
showExtraStyle ? "circle" : "w-3/4 mx-auto mb-5" | |||
showExtraStyle ? "circle" : "w-[280px] mx-auto mb-5" |
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.
any specific for the 280px ?
close keploy/keploy#2159
Issue keploy/keploy#2159
@khareyash05 @Swpn0neel please review PR and merge it.
Now
20241009100015.mp4