Skip to content

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

Merged
merged 9 commits into from
Jun 12, 2024
Merged

Added eye icon #2467

merged 9 commits into from
Jun 12, 2024

Conversation

perminder-17
Copy link
Contributor

@perminder-17 perminder-17 commented Sep 21, 2023

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:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123

@perminder-17
Copy link
Contributor Author

perminder-17 commented Sep 22, 2023

Hi @lindapaiste can u please review the pull request.

@perminder-17
Copy link
Contributor Author

@raclim @lindapaiste can this be merged or should I change something

@perminder-17
Copy link
Contributor Author

I have tested the changes for microsoft edge too. @lindapaiste @raclim

Copy link
Contributor

@PiyushChandra17 PiyushChandra17 left a 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!

@perminder-17
Copy link
Contributor Author

Looks good to me, also working demo seems pretty fine!

Thankyou so much.

@perminder-17
Copy link
Contributor Author

@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?

@raclim raclim added Feature Request Proposal for adding a new functionality Area: Design For UI/UX design updates, proposals, or feedback labels Jan 26, 2024
@katiejliu
Copy link

I think this eye icon looks bette than the other "hide" "show" pull request @raclim

@raclim
Copy link
Collaborator

raclim commented Apr 12, 2024

I think one issue I'm finding here is that the eye icon shifts when the error message for the form appears!

Screenshot 2024-04-12 at 11 33 07 AM

I haven't thought this through yet, but I'm wondering if it'd be better to either move the styles within the LoginForm.jsx and SignupForm.jsx file into a separate one, or to convert it into a styled-component?

@perminder-17
Copy link
Contributor Author

Sorry for the delay, I'll git it done very soon @raclim

@perminder-17
Copy link
Contributor Author

I will be making a commit by june 6th. And will update what's required here.

@perminder-17
Copy link
Contributor Author

p5.js-webe.mp4

so 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.

Copy link
Collaborator

@raclim raclim left a 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! :)

@raclim raclim merged commit 7c50147 into processing:develop Jun 12, 2024
1 check passed
@perminder-17
Copy link
Contributor Author

Hahahhaa no worries @raclim . I love to contribute and will surely contribute in this awesome organization with these awesome Mentainers like you❤

@perminder-17 perminder-17 deleted the eye-icon branch June 12, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Design For UI/UX design updates, proposals, or feedback Feature Request Proposal for adding a new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Eye Icon for Revealing and Hiding Passwords
4 participants