Skip to content

Conversation

@etiennepouliot
Copy link

Does not make sense to use __icontains by default when we have a drop box :

image

Make more sense

image

Also fix searching a BooleanField :

image

Same without a dropbox :

image

Etienne Pouliot added 2 commits December 15, 2021 15:24
@morlandi
Copy link
Owner

Thank you @etiennepouliot
Could you please check my question about changes in filter.py ?

@etiennepouliot
Copy link
Author

@morlandi where is your question ?

@morlandi
Copy link
Owner

@etiennepouliot I'm not very familiar with github code review tools ...
I can see my question right a few lines above this message, and suppose (but I'm not sure) you should see it as well:

Screenshot 2021-12-16 at 15 05 42

or, go to the "File changes" tab of this PR, then scroll down until you see my question just below the pertinent code; here is where I posted it, and you can probably post a reply too:

Screenshot 2021-12-16 at 15 06 26

@etiennepouliot
Copy link
Author

Not sure why, but I can't see your comment.

But no, what I'm really testing is if we have one the letter of 'yes' in the filter so that we can filter for True.

IE , if the user partially type 'ye' if will filter for True, wich is intuitive for the user.

@morlandi
Copy link
Owner

I see ... mumble mumble ... just wondering:
If I type any letter, say "x", nothing is selected, but then if I remove pressing backspace, thus receiving an empty string, then only the True records would be selected, filtering out the False ones ?

@etiennepouliot
Copy link
Author

You're right, nice catch.

Should be better with the last commit
image

image

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