-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Added eye icon #2467
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 eye icon #2467
Conversation
Hi @lindapaiste can u please review the pull request. |
@raclim @lindapaiste can this be merged or should I change something |
I have tested the changes for microsoft edge too. @lindapaiste @raclim |
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.
Looks good to me, also working demo seems pretty fine!
Thankyou so much. |
@lindapaiste Can we merge this PR? Or shall work on it's design? if Yes, can you please tell me what kind of design should i go with? |
I think this eye icon looks bette than the other "hide" "show" pull request @raclim |
Sorry for the delay, I'll git it done very soon @raclim |
I will be making a commit by june 6th. And will update what's required here. |
p5.js-webe.mp4so sorry for being too late @raclim . I was busy with my university exams and GSOC work, happily I got some time to shift styles to seperate one. also fixed the issue which was arising. |
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.
No worries, thanks so much for getting back to this, I really appreciate it!
I made a minor adjustment to the styling of the eye icon (changed the top
property from 5px to 7px
), but overall I think this looks great so far! Thanks so much again for your work on this! :)
Hahahhaa no worries @raclim . I love to contribute and will surely contribute in this awesome organization with these awesome Mentainers like you❤ |
Fixes #2466
Changes: I have made the eye icon which is very helpful. here's how it works
p5.js.Web.Editor._.Login.mp4
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123