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.

Sign up for GitHub

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

Jump to bottom

Search field filters don't work with capital letters #8996

Closed
chdecultot opened this issue May 24, 2017 · 1 comment
Closed

Search field filters don't work with capital letters #8996

chdecultot opened this issue May 24, 2017 · 1 comment
Labels
bug framework

Comments

@chdecultot
Copy link
Contributor

Hi,

When using the POS with the mobile application (iOS or Android) or in a mobile browser, the customer name search box auto filter doesn't work.

It is very inconvenient when looking for a customer to bill.

For example, when entering "Cu" in the search box it should filter on all the names containing "Cu".

Strange thing, it works fine in a laptop browser...
@chdecultot
Copy link
Contributor Author

After further testing, I realized that the issue occurs because mobile devices usually start a new word with a capital letter.

Currently, filtering doesn't work if you enter a capital letter.

The issue is therefore identical in a normal web browser.

I will leave it open, but it is not a high priority to fix I guess.

@chdecultot chdecultot changed the title Customer name auto filter issue on mobile in POS Search field filters don't work with capital letters May 24, 2017
nabinhait pushed a commit that referenced this issue May 26, 2017
@pratu16x7 @nabinhait
[fix] uppercase filter #8996 (#9026)
e38c70c
ashwinisave35 added a commit to ashwinisave35/erpnext that referenced this issue May 29, 2017
@ashwinisave35
Merge branch 'develop' of https://github.com/frappe/erpnext into asse…
d1c0d89 
…t_issue

* 'develop' of https://github.com/frappe/erpnext:
  bumped to version 8.0.40
  [fix] uppercase filter frappe#8996 (frappe#9026)
  Financial Statements Button translateabled (frappe#9032)
  [minor] added get_terms methods to erpnext.utils so that it can be used in non transactional documents (frappe#9037)
  Party Dashboard: Consider any random company if default company not set (frappe#9046)
  [fix] add round off difference to last row in landed cost voucher (frappe#8989)
  Update code to fix issue #erpnext WN-SUP25349 : ValueError: max() arg is an empty sequence. (frappe#9025)
  add milestone field
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug framework
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@chdecultot @pratu16x7 @jwrober