Get all spans in QTableWidget

[JonB]

@Alexey-Serebryakov said in Get all spans in QTableWidget:

What happened? Why 4 ranges? (Each cell as separate range. Why?)

Your question is why your 4 selected cells which form a contiguous "block" are mapped by having 4 separate selection ranges of 1 item rather than 1 selection range of 4 items, is that it?

Well, why not? As QTableWidgetSelectionRange Class says:

The selections in the table may consist of several selection ranges.

Might depend on how you perform the selections (e.g. code vs user interaction)? You use QAbstractItemView::ExtendedSelection. That allows any number of non-contiguous cells to be selected by using Ctrl+click. So maybe it implements multiple selection as individual ones to allow for this, and does not "merge" adjacent ones into a single selection? (You might see how it behaves with, say, QAbstractItemView::ContiguousSelection?) If you need to recognise that pre-process the selected ranges yourself to see if they can be "merged" into fewer, bigger ones?

[Alexey Serebryakov]

@JonB
No, I don't use Ctrl. Just left mouse button. Also I have my toolbar with "Merge cells" button.

But if I use Ctrl+selection then that work correctly. But I don't need so.
Hmm... how can I describe.
I need following:

1. I select a region using only left mouse button. Release left mouse button. Click on my toolbar button to merge selected cells. Cells are merged ok.
2. I doing something with sheet, insert texts in some cells and so on.
3. Select another region of cells using only left mouse button. Release left mouse button. Click on my toolbar button to merge selected cells. Cells are merged ok. (But that don't work. Cells don't merged!).

[Alexey Serebryakov]

@JonB Ok.
How can I span region of some cells if I've already have some spanned cells on the sheet?

[Alexey Serebryakov]

@JonB that is simple functionallity as in MS Excel, MS Word and so on!

[SGaist]

Since you want to enlarge the span, shouldn't you rather calculate the final value and then apply that rather than making all these steps ?

[Alexey Serebryakov]

@SGaist ok but why in my second case QList<QTableWidgetSelectionRange> ranges = selectedRanges(); does not return one of QTableWidgetSelectionRange with top 3, left 4, and span 2, 2?

Here some mistakes qtablewidget.cpp?

QList<QTableWidgetSelectionRange> QTableWidget::selectedRanges() const
{
    const QList<QItemSelectionRange> ranges = selectionModel()->selection();
    QList<QTableWidgetSelectionRange> result;
    const int rangesCount = ranges.count();
    result.reserve(rangesCount);
    for (int i = 0; i < rangesCount; ++i)
        result.append(QTableWidgetSelectionRange(ranges.at(i).top(),
                                                 ranges.at(i).left(),
                                                 ranges.at(i).bottom(),
                                                 ranges.at(i).right()));
    return result;
}

[Alexey Serebryakov]

@SGaist seems to me you don't understand what I need. :-(((

May there is has some flag for table widget?

I need select some cells then click button merge. Select another cells then click button merge. Select another cells click button merge. etc. without using Ctrl and Shift keys.

[SGaist]

Please provide a minimal compilable example so people can test your code and maybe see what's wrong.

[JoeCFD]

@Alexey-Serebryakov Could be a bug since the item indices have been changed from the first merge. You may need to save the initial index set and somehow map the new selection to the initial index set. Can be tricky. What are your OS and Qt version?

[maihai]

@Alexey-Serebryakov I have same proplem , Have you got a solution yet?