New collection filter for (original) image dimensions

[zisoft]

This PR adds a new collection filter for the image dimensions.

Search is performed using wildcards, so one can enter the full dimensions like 6000x4000 or only parts of that:

• 6000x4000 - full match
• 6000x - all images with 6000 width
• x4000 - all images with 4000 height
• ...

closes #21577

[HansBull]

This would benefit from a CREATE INDEX "img_dim" ON "images" (width || 'x' || height)

[zisoft]

@HansBull : good idea, but unfortunately such indexes do not work on LIKE queries.

[HansBull]

@zisoft Sure, and that's true for many other parts of the codebase. But the initial population of the list in misc.c‎ would be snappier. YMMV, though ;-)

[zisoft]

Release notes:

Added a new collection filter for the original image dimensions.

[wpferguson]

@zisoft could you add new collection property to the luaA_enum_values in collect.c please?

[zisoft]

@wpferguson : I always forget this, sorry.

I have added DT_COLLECTION_PROP_DUPLICATES and DT_COLLECTION_PROP_DIMENSIONS.

[wpferguson]

@zisoft that's why you have me 😁

Thanks

[TurboGit]

On my side I don't have this filter:

[zisoft]

???

[TurboGit]

Yep, that's strange... no idea why for now.

[TurboGit]

Sorry I was looking at the collection module and not at the collection filters one :) Will test later.

[TurboGit]

The right-click does not work, on my side when I right click on the entry of the module I get the GNOME default copy/paste actions.

[zisoft]

Very strange. It is just an enhancement of the 'misc' filter. Does that mean that the other misc filters (camera, lens, flash, exposure program, ...) don't work either on your system?

They all use the same code.

Edit: It is a collection FILTER, so your current collection should contain at least some images...

Edit 2: Rebased on current master and it does not work on my system, too.
Most likely the now merged #21659 ?

Edit 3: confirmed, if I drop commit 98d894a it is working again.

[Arecsu]

@zisoft @TurboGit found a bug in event controllers in master, will push a new PR. It is related to this

And then one to your branch @zisoft to port your code to controllers :)

[Arecsu]

Here is the PR #21718

@zisoft after this one lands master, rebase this PR and you should be ready to go. I thought your code needed some wiring but no, it's fine.

#21659 converted the misc filter entries to event controllers, but the gesture only listened for the primary button, so right-click on any misc filter (camera, lens, flash, …) showed the default cut/copy menu instead of the "select existing values" popup.

[TurboGit]

Merging now. Thanks!