Update `key` computation logic in `$addToRecentSearchesFn`

[babblebey]

The current logic in the $addToRecentSearchesFn function of /lib/stores/search.js computes the key for recent searches by converting the word to lowercase and replacing spaces with hyphens. This can be improved by using the normalizeAsUrl utility function exported from lib/utils/index.js to ensure consistency and better handling of special characters.

Proposed Changes:

Update the key computation in $addToRecentSearchesFn from:

const lowercaseKey = word.toLowerCase();
const key = lowercaseKey.includes(" ") ? lowercaseKey.split(" ").join("-") : lowercaseKey;

to:

const key = normalizeAsUrl(word);

Related Files:

• https://github.com/jargonsdev/jargons.dev/blob/main/src/lib/stores/search.js

Additional Information:

• Nothing much, feel free to ask any question if you need more clarity on the steps to take 😉

[chiomalovet]

Hi i would love to work on this

[babblebey]

Have at it @chiomalovet 😉

[chiomalovet]

when pushing am i pushing to main or i am creating a new branch?

[babblebey]

when pushing am i pushing to main or i am creating a new branch?

Hey @chiomalovet

You could push to main directly from your fork, but it's not a good practice, because you might want to keep a sync-able copy of the project's defaults on your account too.

So, here's steps to follow.

• Fork the project (Sure you've done this)
• Clone the fork to your machine and run it locally (You done that too)
• Then Create a Branch where you will make your changes and push (this is what you do next 👈🏾)

As soon as you're done with and pushed your changes, come back to the repo here, you will already see a prompt advising you create a pull request against that branch you created.