Releases: Financial-Times/n-feedback
Releases · Financial-Times/n-feedback
Style improvements
https://jira.ft.com/browse/AT-2096
This versian adress the following change request from the Jira ticket:
- Mobile, spacing of text and button
- Popup, "here" needs to be underlined to indicate hyperlink
Further tweak to load performance
https://github.com/Financial-Times/n-feedback/releases/tag/v4.3.0 added support for deferred loading to n-feedback, loading the survey only when the UI is interacted with; this is a further small tweak deferring overlay creation to the same interaction instead of on load.
Remove not supported ES6 spread syntax
This caused building issues on n-ui.
Add performance optimisation
Load survey when n-feedback is clicked and show spinner while the form is loading.
Customise data what we pass from init and what we forward to qualtrics.
Fix handling for missing survey
Merge pull request #43 from Financial-Times/fix-arg-and-check-bugs Fix missing arg and checking bugs
Fix overlay targeting
Merge pull request #41 from Financial-Times/fix/overlay-targeting Target overlay with custom feedback class
Multi instance rendering
For App integration I had to prepare the component to render on multiple page due to the fact, App handles page swiping with a Gallery mecahnism.
Include missing styles
Merge pull request #36 from Financial-Times/fix/missing-styles Include necessary form styles
Fix Edge FormData bug
Bumping to a major version
- N-ui's bower.json was linking to "n-feedback": "^1.0.0-beta.10"
- https://github.com/Financial-Times/n-ui/blob/master/bower.json#L32
- It turns out that you can't pin to a range of pre-releases
- So ^1.0.0-beta.10 resolves to v1.2.3
- Version v1.2.3 is over two years old. Hence the calamity.