Project

General

Profile

Bug #12502

2.6.0 Multiple selections issue

Added by Jean-Luc Lacroix 12 months ago. Updated 12 months ago.

Status:
Fixed
Priority:
Low
Assignee:
Category:
Lighttable
Target version:
Start date:
12/27/2018
Due date:
% Done:

100%

Estimated time:
Affected Version:
git stable branch
System:
all
bitness:
64-bit
hardware architecture:
amd64/x86

Description

Multiple selections in LT

Example:
select pictures #10 and pictures from #15 to #20

Expected behavior (as in 2.4.4):
Click on first picture (picture #10), second ctrl-click to select picture #15 and extend that selection with a shift-click on picture #20

Actual behavior in 2.6.0
On the selection extension (shift-click) on picture #20 all pictures from picture #10 to #20 will be selected instead of only having pictures #10 and #15 to #20 being selected

Also reported on MacOS on the mailing list (date 2018-12-26)

Associated revisions

Revision 917cb519 (diff)
Added by Pascal Obry 12 months ago

Fix multi-selection with range.

Fixes #12502.

Revision b09ba253 (diff)
Added by Pascal Obry 12 months ago

Fix multi-selection with range.

Fixes #12502.

History

#1 Updated by Jean-Luc Lacroix 12 months ago

Multiple selections in LT

Example:
select picture #10 and pictures from #15 to #20

Expected behavior (as in 2.4.4):
Click on first picture (picture #10), second ctrl-click to select picture #15 and extend that selection with a shift-click on picture #20

Actual behavior in 2.6.0
On the selection extension (shift-click) on picture #20 all pictures from picture #10 to #20 will be selected instead of only having pictures #10 and #15 to #20 being selected

Also reported on MacOS on the mailing list (date 2018-12-26)

#2 Updated by Pascal Obry 12 months ago

  • % Done changed from 0 to 100
  • Assignee set to Pascal Obry
  • Status changed from New to Fixed

Should be fixed, see https://github.com/darktable-org/darktable/commit/917cb51973136239ca77122b7913251e5411da83

If you can test and report if it works on your side it would be great. Then we'll merge for 2.6.1. Thanks.

#3 Updated by Pascal Obry 12 months ago

  • System changed from Debian to all
  • Target version changed from 2.6.0 to 2.6.1

#4 Updated by Jean-Luc Lacroix 12 months ago

Patch applied and tested. Problem solved. Thanks.

#5 Updated by Pascal Obry 12 months ago

Thanks for testing.

Also available in: Atom PDF

Go to top