-
Notifications
You must be signed in to change notification settings - Fork 5
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
Redesign homepage #488
Redesign homepage #488
Conversation
Search bar now gets results on keyup
…h bar. Urls now come from config files.
# Conflicts: # src/app/modules/home/highlight-slider/highlight-slider.component.ts # src/app/modules/home/home.module.ts
…nto feature/380-redesign-homepage
I only changed the highlight logic, no styling. @liekevdvoort was going to look at it iirc. |
…n' into feature/403-autocomplete-searchresult-newhomepage
…hows on the homepage for now. Also limited search results to 5
…load in when the user is authenticated and when there are 2 or more recommendations to show
…t. Conditional rendering on home page based on authentication
… also show up if there is only 1 recommendation.
….com/DigitalExcellence/dex-frontend into feature/497-recommendations-homepage
…ndations-homepage
…tions-homepage Recommendations on homepage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Well done 🎉
Description
Implementation of the new homepage designs
Type of change
Checklist
Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
These steps will be used during release testing.
Link to issue
Closes: #380