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

Added neuromorphism effect #20

Closed
wants to merge 1 commit into from

Conversation

Whitedevil1206
Copy link

@Whitedevil1206 Whitedevil1206 commented Dec 26, 2020

Closes #7
Pls review this PR .

  • I have checked it on my local pc and its working fine.

@tusharnankani
Copy link
Owner

tusharnankani commented Dec 26, 2020

Hi @Whitedevil1206
Could you attach a demonstration of how it looks like? You can share screenshots or record your screen. You can directly drag .mp4 files in here.

Or even better, host your forked repository on Netlify and send the link here. Let me know if you need any help!

@tusharnankani tusharnankani added the enhancement New feature or request label Dec 26, 2020
@Whitedevil1206
Copy link
Author

Ok, i will send it tomorrow

@Whitedevil1206
Copy link
Author

Whitedevil1206 commented Dec 27, 2020

Hello this is the netlify link :

Copy link
Owner

@tusharnankani tusharnankani left a comment

Choose a reason for hiding this comment

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

Hi @Whitedevil1206

Good work 👍🏻
Apologies for the late review, was occupied with some commitments.

Also, with all due respect, after your sincere efforts, the end product might not look as good as we thought it would. Maybe, this to-do list web page is supposed to look simple after all, and neomorphic might not look as expected after all. But all that matters is trying and it counts.

Now, I would request you to play around more with neomorphic effects - since the website doesn't really look as appealing as we thought it would.

neomorphism-feedback.mp4
  • Play around with effects on all 3 buttons, try adding hover, and in-focus effects.

  • As you have added a hover on input, you could add some inset, offset hover effects on the todos as well.

  • Also here, could you try changing the color theme? The neomorphic effect doesn't really go well!
    image

Appreciate your efforts! 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a Neuromorphic Interface
2 participants