[KPhotoAlbum] [patch] Excluding Files and Directories

[Jesper K. Pedersen]

On Wednesday 27 September 2006 04:19, Ashley J Gittins wrote:
| > Thanks for the patch. I attached a new patch for you which compiles with
| > current trunk.
|
| Ahh, thanks. I think I missed a few things trying to move it to trunk,
| sorry about that.
|
| > Though I agree this might solve some people problems, I really doubt that
| > the majority of KPA users would know how to create a regexp (even with my
| > editor), which exclude files in certain directories and with certain
| > extension.
|
| Agreed - to be honest, I knew it was a bit of a stretch expecting the
| average user to grok regexps, and seeing the editor would probably just
| make it look harder again :-)
|
| Getting the feature started and under discussion was motivation enough
| though to push it out for comment anyway.
|
| > Any suggestion for Ashley, for improvements?
|
| I have one :-)
| How about doing something very similar to how Kate's "Find" dialog works?
| Default to using a wildcard match so that users can enter "*.NEF" or
| "backup*", keeping things simple. Then under that a checkbox labelled
| "regular expression" and the button to launch the regexpEditor. The button
| will remain disabled as long as the checkbox is clear.
How about splitting up into files and directories?

| It shouldn't take too much more to add a setting for ignoreRegexpSyntax
| and associated checkbox, plus (optionally?) a button to launch the regexp
| editor.
|
| Part of this will also be to add regexp validation to prevent entering
| invalid regexps, if regexps are enabled.
|
| Hopefully this would give the balance between user-friendlyness (wildcards
| by default) and power (regexps if desired).
Sounds good to me.

| > Ashley, before I want to include this, I really need it a bit more
| > polish, hope you understand.
|
| Completely! Part of my motivation was to get a stick-in-the-sand so to
| speak, to ensure that the solution to several problems was met with one
| fix that makes a powerful feature. Refining that fix is all good,
| providing we can keep the power of it while making it easy enough for any
| user to take advantage of.
|
| Further comments welcome.
OK, hope you will adjust your patch for this, then I'm sure we will get it in.

Cheers
Jesper.