[Development] PROPOSAL: QtWidgets: Add setRowVisible and setColumnVisible to QTableView

[Thorbjørn Martsum]

Hi Roman

Thanks for your suggestion and your interest in improving Qt. If QTableView
hadn't already had hideRow/showRow and hideColumn/showColumn - I would
however hesitate with supporting them added. I do not see the problem with:
table->verticalHeader()->setSectionHidden (QHeaderView also hase
hideSection, showSection).

The tableView functions have however been added for convenience. The reason
there is no single function that takes an argument is that they are slots,
and when connecting signals it has been considered more easy to have it
that way. It wouldn't be easy to connect something with row x happened so
it should be hide the row with a slot that takes a boolean, too.

*OK, that means the problem is in docs: there are no references to those
methods from (show/hide)(Row/Column). How could I find it if methods don’t
copy QWidget’s logic and even descriptions of methods don’t contain word
“visible”?
*
I think you are reading the documentation a bit wrong. Read (all) class
member-functions to understand it. The functions exist and they are
well-documented (more than other things, actually). I don't know if it is
possible to tweak in the word visible without making the documentation
sound weird, but if you have any suggestion to making documentation better
- feel free to upload them to gerrit (codereview.qt-project.org) or discuss
it with somebody on qt-labs.

regards
Thorbjørn

On Mon, May 20, 2013 at 6:11 PM, Roman Inflianskas <infroma at gmail.com>wrote:

> As Chris Kawa pointed in thread
> <http://qt-project.org/forums/viewthread/27971/>there are methods
> QTableView::setRowHidden<http://qt-project.org/doc/qt-5.0/qtwidgets/qtableview.html#setRowHidden>
> *,*QTableView::setColumnHidden<http://qt-project.org/doc/qt-5.0/qtwidgets/qtableview.html#setColumnHidden>
> .
> OK, that means the problem is in docs: there are no references to those
> methods from (show/hide)(Row/Column). How could I find it if methods don’t
> copy QWidget’s logic and even descriptions of methods don’t contain word
> “visible”?
> But: why this methods are inverted? I think it would be more natural if
> they be setRowVisible/setColumnVisible.
>
>
> On Mon, May 20, 2013 at 7:53 PM, Roman Inflianskas <infroma at gmail.com>wrote:
>
>> Hi.
>> I want to change visibility of rows and columns of QTableWidget. I know,
>> that I can use (show/hide)(Row/Column) methods, but sometimes more elegant
>> solution is to use functions that take visibility factor. So, I think it
>> would be better to have methods setRowVisible/setColumnVisible. Second
>> reason is current inconsistency: QWidget and other classes provide all this
>> methods: show, hide, setVisible, while QTableView doesn’t have similar
>> methods.
>> I think that this wouldn’t break API.
>>
>> --
>> Regards, Roman Inflianskas
>>
>
>
>
> --
> Regards, Roman Inflianskas
>
> _______________________________________________
> Development mailing list
> Development at qt-project.org
> http://lists.qt-project.org/mailman/listinfo/development
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/development/attachments/20130520/e8236900/attachment.html>