search

jncraton / course-search

1 stars 0 forks source link

Search bar experiment #87 #92

Closed shama2025 closed 1 year ago

shama2025 commented 1 year ago

87

netlify[bot] commented 1 year ago

Deploy Preview for cpsc4430-course-search ready!

Name Link
Latest commit a8cf397dca683ed1dcf0614c2f87cc0724671705
Latest deploy log https://app.netlify.com/sites/cpsc4430-course-search/deploys/6508c2ae33cbf300095a9e41
Deploy Preview https://deploy-preview-92--cpsc4430-course-search.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

scoutskylar commented 1 year ago

The final line in main.js should be changed to either updateTable() or updateTable(''). Currently, it causes the initial table load to fail.

scoutskylar commented 1 year ago

I changed the placeholder text to match the column headings and moved the code that gets the search query from the input element inside the updateTable function, since the previous method was being overwritten by changes to the filter. For some reason, Prettier on my device is not following the same code styling as this project, so someone else will need to run it.

shama2025 commented 1 year ago

Closed #87