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

Loading indicator in theme #875

Merged
merged 2 commits into from
Jul 8, 2021
Merged

Loading indicator in theme #875

merged 2 commits into from
Jul 8, 2021

Conversation

yen-tt
Copy link
Contributor

@yen-tt yen-tt commented Jul 2, 2021

  • add opacity setting in theme css
  • add default loadingIndicator to true in searchbar

Note: loading indicator changes require corresponding loading indicator udpates from SDK

J=SLAP-1335
TEST=manual

  • Loading indicator is present when set to true during loading
  • During loading, universal result section have opacity set to 0.5 and 1. Confirm visual update on page

- add opacity setting in theme css
- add default loadingIndicator to true in searchbar

Note: loading indicator changes require corresponding loading indicator udpates from SDK

J=SLAP-1335
TEST=manual
@coveralls
Copy link

coveralls commented Jul 2, 2021

Coverage Status

Coverage remained the same at 6.765% when pulling fe541b4 on dev/loading-indicator into 43c64e9 on develop.

@yen-tt yen-tt merged commit aa3fffd into develop Jul 8, 2021
@yen-tt yen-tt deleted the dev/loading-indicator branch July 8, 2021 14:50
@cea2aj cea2aj mentioned this pull request Aug 24, 2021
cea2aj added a commit that referenced this pull request Aug 24, 2021
### Features
- Support for new languages including Chinese (Traditional), Chinese (Simplified), Russian, Polish, Portuguese, Dutch, Arabic, Korean, Swedish, and Hindi (#918) (#900)
- A loading indicator can now be enabled on the search bar (#875)
- Voice search support (#894)
- Analytics, session tracking, and query source can now be toggled through Runtime Config (#869) (#879)
- Added validation to ensure that parts of global_config and page configs are valid (#888)
- Font preloads are now located in the layouts/preload-fonts.hbs file (#893)
- Global config now includes "sessionTrackingEnabled" and "analyticsEventsEnabled" (#898)
- Added a partial for additional Answers configuration (#903)
- Map pins now support custom label colors (#922)

### Enhancements
- The full-page-map "Search This Area" functionality is better optimized when a user quickly zooms in or out of the map (#861)
- Improved build time during development mode (#865)
- Icons were updated from components to Handlebars partials to improve performance (#874)

### Bugfixes
- When upgrading the theme version, the global config will no longer contain duplicate options which are commented out (#916)
- When upgrading the theme, the tests folders are now removed which improves build performance (#913)
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.

4 participants