Skip to content

Bug fixes#177

Merged
rosemcc merged 13 commits into
masterfrom
bug-fixes
Nov 24, 2021
Merged

Bug fixes#177
rosemcc merged 13 commits into
masterfrom
bug-fixes

Conversation

@rosemcc
Copy link
Copy Markdown
Contributor

@rosemcc rosemcc commented Nov 23, 2021

This PR includes the following:

  • capitalised the manually curated search autocomplete terms
  • cleaned up unused global variables
  • custom 500 error message
  • provide fallback storage options for localforage
  • numbered lists on safari now show the numbers
  • reduced hub logo size on smaller screens
  • fixed scrolling image bug on safari mobile on home page
  • adds a cron job to run the search proxy bulk function on a daily schedule

@rosemcc rosemcc requested a review from Trombach November 23, 2021 02:55
Comment thread research-hub-web/src/app/components/home/banner-image/banner-image.component.html Outdated
Copy link
Copy Markdown
Contributor

@Trombach Trombach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Capitalised autocomplete is a lot less jarring, at least for my eyes

@rosemcc rosemcc merged commit 15f645e into master Nov 24, 2021
@rosemcc rosemcc deleted the bug-fixes branch November 24, 2021 21:49
@rosemcc rosemcc mentioned this pull request Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants