[BUG-FIX] "Recent Searches" list doesn't work because of changed result list structure

[icyflame]

Bug

1. Clicking on a result link doesn't add the current search term to the recent searches link
2. Even if new searches are added, old searches are not removed properly leading to duplicates in the Recent Searches list (even though there are none in Local Storage)

Trigger

#52 made a lot of changes to the UI and the JS code.

1. Recent searches not working
   1. Document click handler added the search to the recent searches list
   2. This handler was checking to see if the clicked element had the class result-link
   3. The result list structure was changed from a.result-link to a.result-link > div h1
   4. In the new structure, every click's target would be one of the children: div or h1, and neither of them have the result-link class, so the click handler would not consider the clicked element as a result link and won't add the search term to the recent searches list
2. Duplicate searches not removed from the Recent searches list
   1. Once bug 1 is fixed, this bug shows up. This was also introduced by the same PR
   2. When a new search term is clicked, the current logic performs the following steps:
      1. Replace current term it in the old stored string
      2. Add current term to the old string
      3. Store the string in localStorage
      4. Remove the elements matching the current term from the currently displayed Recent Searches list => BUG
      5. Prepend the current search term to the recent searches list (thus bringing to the top)
   3. In step 4 above, the removal relies on the structure of the Recent Searches list.
      1. Before: ul.local-storage-list div > a
      2. After: ul.local-storage-list > li > a

Impact

• Recent searches were not stored for users since Oct 19, 2019

Fix

1. Attach the handler to a.result-link => This ensures that the handler is triggered whenever the a or one of it's children are clicked
2. Fix the element pattern for removing duplicates from the recent searches page

I have tested this locally in a bunch of scenarios and it works as expected. 👍

Fix #54

Other changes in this PR

• Make the UI UX Mobile friendly and refreshing #52 also converted several function calls to write the arguments one line each. This is especially confusing when the full search call can be accomodated on a single line. I changed some fuctions. I will run semistandard --lint on the JS file in a later PR.

[thealphadollar]

LGTM. Thanks a lot for taking out time from your schedule to send the fix ✨

A small change I've requested, please see if it fits. I'll merge it post your response.