https://bugs.kde.org/show_bug.cgi?id=375885

Boudewijn Rempt <b...@valdyas.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/kde/ |https://invent.kde.org/kde/
                   |krita/commit/2568bb5825b748 |krita/commit/0f00ae703c853b
                   |d569cbb75c228d6f68e690d0ba  |ca2d699cd2ee95e50dc0ed8120

--- Comment #4 from Boudewijn Rempt <b...@valdyas.org> ---
Git commit 0f00ae703c853bca2d699cd2ee95e50dc0ed8120 by Boudewijn Rempt.
Committed on 10/09/2019 at 08:25.
Pushed by rempt into branch 'krita/4.2'.

Fix sort order of frames with simple naming scheme

QCollator::isNumerical works in platform-specific and Qt-version
specific ways, and cannot be guaranteed to sort mixed alphanumerical
strings in a numerical way. It just doesn't work that way.

This patch extracts all numbers for the comparant strings, and then
checks the last number in each string numerically. Now, if the user
would make frames like this:

scene1frame9
scene1frame10

Everything is fine.

frame9scene1
frame10scene1

will break; and unless we offer a kind of filename templating thing
where the user can specify which part of the filename provides the
frame number, there's nothing more we can do.

M  +15   -4    libs/ui/dialogs/kis_dlg_import_image_sequence.cpp
M  +5    -1    libs/ui/forms/wdgimportimagesequence.ui

https://invent.kde.org/kde/krita/commit/0f00ae703c853bca2d699cd2ee95e50dc0ed8120

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to