ReVanced / revanced-website

🌐 Website for ReVanced
https://revanced.app
GNU General Public License v3.0
313 stars 42 forks source link

Changes required in the "Search for patches" placeholder in patches tab #224

Closed RiddhimannM closed 5 months ago

RiddhimannM commented 10 months ago

Listing down few issues/enhancement ideas for the search for patches placeholder in the patches path based on my observations as an end-user.

PS - This is my first issue as an open source contributor, so please excuse me for mistakes if any :)

https://github.com/ReVanced/revanced-website/assets/84954173/b7552c85-622f-4da4-8715-fac7d1cd9af6

https://github.com/ReVanced/revanced-website/assets/84954173/adc50a10-9e9d-47e3-9cd3-a5b2fae6ebcf

oSumAtrIX commented 10 months ago
  1. It looks like a string sanitization problem. In older times the patch names were stripped of the "-" character, so that may be the cause.
  2. This would need some semantical changes such as moving the search from the header to the content
oSumAtrIX commented 5 months ago
  1. is fixed, please reopen an issue for 2.