Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix browser nav & remove the need to double-ENTER on the searchbar #298

Merged
merged 29 commits into from
Nov 21, 2024
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
3832725
make back nav more intuitive (every time you click search it adds to …
AbhiramTadepalli Oct 31, 2024
e9e6e10
Dynamically change page title based on searchTerms. Helps make back n…
AbhiramTadepalli Nov 1, 2024
57f4566
lint check
AbhiramTadepalli Nov 1, 2024
1ce794d
Trigger new search when clicking ENTER to select add an autocomplete …
AbhiramTadepalli Nov 2, 2024
c93e6a9
Fix formatting of the header (trailing comma, add 'Results - ' at the…
AbhiramTadepalli Nov 2, 2024
673bbf4
Remove state
TyHil Nov 8, 2024
11e2a2f
Bring the meta property for link preview into dashboard/index.tsx so …
AbhiramTadepalli Nov 8, 2024
9dc3e06
(revert later) add a test to diagnose link preview bug
AbhiramTadepalli Nov 8, 2024
17f3293
Revert "Remove state"
AbhiramTadepalli Nov 8, 2024
143f080
Reapply "Remove state"
AbhiramTadepalli Nov 8, 2024
a782977
Revert "(revert later) add a test to diagnose link preview bug"
AbhiramTadepalli Nov 8, 2024
96afc30
try to directly make the meta tag from the url
AbhiramTadepalli Nov 10, 2024
357a222
Merge branch 'develop' of https://github.com/UTDNebula/UTD-Trends int…
AbhiramTadepalli Nov 10, 2024
25280ab
small changes based on inline comments
AbhiramTadepalli Nov 19, 2024
e26529b
testing SSRajmouli for link previews
AbhiramTadepalli Nov 19, 2024
9557009
Try to enable dynamic page titles alongside SSR
AbhiramTadepalli Nov 20, 2024
0bdbf6c
Try again to enable dynamic page titles alongside SSR
AbhiramTadepalli Nov 20, 2024
45dd32f
Revert "Try again to enable dynamic page titles alongside SSR"
AbhiramTadepalli Nov 20, 2024
9ac1e75
Add a dynamicPageTitle state that is initialized to the SSR-generated…
AbhiramTadepalli Nov 20, 2024
543d7e3
order searchterms properly in link preview (courses first and then pr…
AbhiramTadepalli Nov 20, 2024
6aed954
c'mon I forgot linting
AbhiramTadepalli Nov 20, 2024
47ecdb5
FORMAAAAAT
AbhiramTadepalli Nov 20, 2024
7b2aeef
i figured it out maybe
AbhiramTadepalli Nov 20, 2024
47384d2
refactor: moved search term processing to a function as it is called …
AbhiramTadepalli Nov 20, 2024
760af45
format and lint
AbhiramTadepalli Nov 20, 2024
b0f55e2
refactor: remove a dynamicPageTitle state; build the page title via f…
AbhiramTadepalli Nov 20, 2024
d50301c
Update description meta tag
AbhiramTadepalli Nov 20, 2024
99ac32e
remove redundant onChange_internal() in updateValue()
AbhiramTadepalli Nov 20, 2024
932d252
Merge branch 'develop' into fix-browser-nav
AbhiramTadepalli Nov 21, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/components/common/SearchBar/searchBar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,7 @@ const SearchBar = ({
if (newValue.length) setErrorTooltip(false); //close the tooltip if there is at least 1 valid search term
setValue(newValue);
onChange_internal(newValue);
AbhiramTadepalli marked this conversation as resolved.
Show resolved Hide resolved
onSelect_internal(newValue); // clicking enter to select a autocomplete suggestion triggers a new search (it also 'Enters' for the searchbar)
AbhiramTadepalli marked this conversation as resolved.
Show resolved Hide resolved
TyHil marked this conversation as resolved.
Show resolved Hide resolved
}

//update parent and queries
Expand Down
Loading