Thanks.

This patch doesn't seem to contain the logic required for what you demonstrated in the video. It's only about the calculation of properties. Did you forget to add some other changes to this patch?

Please be aware of our code style which is documented src/doc/coding_style.dox. The relevant parts for this patch are:

• we don't use any brackets for one-line if-else statements
• we use spacing after 'if' and before the opening bracket - if (condition) {...

You can also check the file admin/READMY.astyle to see how to quickly re-format the code according to LabPlot's code style.

we mostly used the camel case naming convention. You seem to mix different conventions like in xdistsquare (instead of xDistSquare) and lower_index(instead of lowerIndex). Can you please unify the naming?

Furthermore, we'll need to extend this logic to datetime data type where we store QDateTime instead of doubles in the column. But let's finalize this patch firs and extend/adjust the logic after that.

Can the code:

    //to select curves having overlapping bounding boxes we need to check whether the cursor
    //is inside of item's shapes and not inside of the bounding boxes (Qt's default behaviour).
    if(m_cartesianPlotMouseMode == CartesianPlot::MouseMode::SelectionMode){
		for (auto* item : items(event->pos())) {
			if (!dynamic_cast<XYCurvePrivate*>(item))
				continue;

			if ( item->shape().contains(item->mapFromScene(mapToScene(event->pos()))) ) {
				//deselect currently selected items
				for (auto* selectedItem : scene()->selectedItems())
					selectedItem->setSelected(false);

				//select the item under the cursor and update the current selection
				item->setSelected(true);
				selectionChanged();

				return;
			}
		}
    }

In worksheetView.cpp, line944 to 963 be deleted?

Thanks in advance for fixing this minor issue.