Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

fix: Pressing enter when searching no longer goes to about:blank #504

Closed
wants to merge 1 commit into from

Conversation

adnanjpg
Copy link

@adnanjpg adnanjpg commented Jan 5, 2024

Description

this pr fixes #501

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

fixes #501

Mobile & Desktop Screenshots/Recordings

Steps to QA

try to press enter in the search repositories field, before the repos load

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@adnanjpg adnanjpg marked this pull request as ready for review January 5, 2024 18:14
Copy link

netlify bot commented Jan 5, 2024

βœ… Deploy Preview for hot-sauced-ui ready!

Name Link
πŸ”¨ Latest commit 553a7d1
πŸ” Latest deploy log https://app.netlify.com/sites/hot-sauced-ui/deploys/659847177416e7000a7f0b24
😎 Deploy Preview https://deploy-preview-504--hot-sauced-ui.netlify.app/
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance:
Accessibility:
Best Practices:
SEO:
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@adnanjpg adnanjpg marked this pull request as draft January 5, 2024 18:30
@adnanjpg
Copy link
Author

adnanjpg commented Jan 5, 2024

one more thing around this issue, is that the user can't just navigate to a repository by just clicking on it in the result list, the user has to click on it to set it as selected then has to press enter on the field, should I work on a fix for that here or with a new issue & pr?

@nickytonline nickytonline changed the title Search repos about blank fix: Pressing enter when searching no longer goes to about:blank Jan 6, 2024
@bdougie
Copy link
Member

bdougie commented Apr 1, 2024

@adnanjpg checking in here. Let us know if you are still around to work on this.

one more thing around this issue, is that the user can't just navigate to a repository by just clicking on it in the result list, the user has to click on it to set it as selected then has to press enter on the field, should I work on a fix for that here or with a new issue & pr?

Open a new issue.

@bdougie
Copy link
Member

bdougie commented Apr 16, 2024

#506 (comment)

@bdougie bdougie closed this Apr 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: search repositories pressing enter takes to about:blank
2 participants