p5.js-web-editor icon indicating copy to clipboard operation
p5.js-web-editor copied to clipboard

issue #3241

Open Harshit-7373 opened this issue 1 year ago • 1 comments

Icon Alignment Issue in Password Field – Adjust Eye Icon Vertical Centering #3241

github-7373

Fixes #issue-number

Changes:

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

Harshit-7373 avatar Oct 22 '24 14:10 Harshit-7373

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already.

welcome[bot] avatar Oct 22 '24 14:10 welcome[bot]

Thanks for giving this a look!

There's a few issues going on with this pull request, but I feel like one larger one here is that it includes a CSS file rather than a SASS one, which is predominantly used throughout the repository. This causes the branch to not load, which for me, indicates that you weren't able to verify the submitted changes. We strongly encourage ensuring that your changes work, pass all the criteria listed in the checkboxes, and provide a working example (such as screenshots or videos) so your pull request is ready to go once you've submitted it.

I'm going to close this for now, but would encourage you to try to give this another try! If you decide to give this another look, I also feel like that adding an another style sheet probably won't be needed here, as this portion is already being handled in _forms.scss, so that would be a good starting point!

raclim avatar Nov 01 '24 16:11 raclim