Skip to content

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