Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix(searchbox): add maxlength 512 #542

Merged
merged 6 commits into from
Nov 2, 2017
Merged

fix(searchbox): add maxlength 512 #542

merged 6 commits into from
Nov 2, 2017

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Oct 31, 2017

Summary

The API can only handle queries up to 512 characters and would throw an error in this case (copy pasted addresses for example).

Result

This prevents users from typing more than possible, and would simply discard later characters

fixes #510 (haha would be funny is that was issue 512)

supported in all browsers, but would fall back nicely anyway: http://caniuse.com/#feat=maxlength

cc @67ideas

The API can only handle queries up to 512 characters and would throw an error in this case (copy pasted addresses for example). This prevents users from typing more than possible, and would simply discard later characters

fixes #510 (haha would be funny is that was issue 512)

supported in all browsers, but would fall back nicely anyway: http://caniuse.com/#feat=maxlength

cc @67ideas
@algobot
Copy link
Contributor

algobot commented Oct 31, 2017

Deploy preview ready!

Built with commit fb98095

https://deploy-preview-542--react-instantsearch.netlify.com

@mthuret
Copy link
Contributor

mthuret commented Oct 31, 2017

Apparently you should use maxLength @Haroenv :)

Copy link
Contributor

@mthuret mthuret left a comment

Choose a reason for hiding this comment

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

LGTM

@mthuret mthuret merged commit 5bd4033 into master Nov 2, 2017
@mthuret mthuret deleted the searchbox/maxlength branch November 2, 2017 12:59
samouss added a commit that referenced this pull request Nov 2, 2017
<a name="4.2.0"></a>
# [4.2.0](v4.1.3...v4.2.0) (2017-11-02)

### Bug Fixes

* **connectRange:** handle boundaries on first call ([9f14dc0](9f14dc0))
* **connectRange:** use refine instead of cleanUp in metadata ([#526](#526)) ([1861235](1861235))
* **hierarchicaMenu:** allow sorting and using limit ([fe178ed](fe178ed)), closes [#92](#92)
* **InfiniteHits:** add disabled style to the LoadMore button ([#477](#477)) ([faba1ad](faba1ad))
* **Range:** handle float, allow reset and respect boundaries ([75969b8](75969b8))
* **RangeInput:** fix compatibility with React 16 & Panel ([3f218db](3f218db))
* **searchbox:** add maxlength 512 ([#542](#542)) ([5bd4033](5bd4033)), closes [#510](#510)

### Features

* **MenuSelect:** add component and connector ([cc6e0d7](cc6e0d7))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Searchbox: limit to 512 characters
3 participants