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

Find in files exclude as a textbox is very limited (not very UX friendly either) #4474

Closed
goanpeca opened this issue May 15, 2017 · 4 comments

Comments

@goanpeca
Copy link
Member

goanpeca commented May 15, 2017

(Side note: This \.py\.orig$ makes sense as an extra default for find in files produced by git when rebasing merging)

The issue with the exclude textbox is that it is very limited as more and more things are added. I think exclude should be a two column table, where each row had one entry and the column had a description, so that users can add comments to specific patterns of exclusion. Also this makes the list/table usable when we start adding more exclusion patterns. Also the exclusion patterns right now are python based, moving towards python agnosticism will require more and more patterns and this will make the lineedit unusable.

@goanpeca goanpeca changed the title Find in files exclude Find in files exclude as a textbox is very limited (not very UX friendly either) May 15, 2017
@andfoy
Copy link
Member

andfoy commented May 15, 2017

The table should include also a checkbox that indicates if a pattern is active or not?

@goanpeca
Copy link
Member Author

The table should include also a checkbox that indicates if a pattern is active or not?

Could be?

@ccordoba12 ccordoba12 added this to the v4.0beta2 milestone May 15, 2017
@ccordoba12
Copy link
Member

I think we could add a Preferences entry for Find in Files to solve this (and also a config entry for Projects)

But this is not something for 3.2 but for Spyder 4.

@ccordoba12 ccordoba12 modified the milestones: v4.0beta2, v4.0beta3 Aug 13, 2017
@ccordoba12 ccordoba12 modified the milestones: v4.0betaX, v3.3.2 Sep 9, 2018
@ccordoba12
Copy link
Member

We added a simpler way to exclude files in 3.x and this is going to be applied to the file names only. I doubt we're going to do something more about this in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants