Differential D2035

Fix T47441: Allow image open operator to load multiple images at once
ClosedPublic
Actions

Authored by jside on May 28 2016, 9:01 PM.

Details

Reviewers

Maniphest Tasks

T47441: Image Open: support for opening multiple images at once

Summary

This patch allows the image open operator to load multiple images with the last one given as by the file list
being passed as the active one.

For image sequences it will only load the first image and populate the ImageUser struct accordingly as before. Duplicates are not loaded twice but if the active image is already loaded it's already existent instance will be passed on as the active image.

Diff Detail

Repository: rB Blender

Event Timeline

jside updated this revision to Diff 6807.May 28 2016, 9:01 PM

jside retitled this revision from to Fix T47441: Allow image open operator to load multiple images at once.

jside updated this object.

jside added a reviewer: Campbell Barton (campbellbarton).

jside set the repository for this revision to rB Blender.

jside added a task: T47441: Image Open: support for opening multiple images at once.

Just tried it, it looks like it works!

Updated this patch and added support for mixing multiple images and frame-ranges. P363

A general issue with this behavior (noticed just now) is not giving the newly loaded images a user means they will be removed on re-saving.
This may seem like a bug to users.
So question is weather this is acceptable behavior, I'm not keen on it.
(as normal, quick hacks expose design issues!)

Campbell Barton (campbellbarton) accepted this revision.Jul 15 2016, 8:03 AM

Campbell Barton (campbellbarton) edited edge metadata.

This revision is now accepted and ready to land.Jul 15 2016, 8:03 AM

Campbell Barton (campbellbarton) mentioned this in rBccee251a40f9: Support for loading multiple images from the file selector.Jul 15 2016, 8:03 AM

Committed P363 rBccee251a40f985492cffe3eac905a6b50e751d72

Brecht Van Lommel (brecht) mentioned this in T47441: Image Open: support for opening multiple images at once.Oct 8 2016, 6:15 PM