Hi

I would really like to welcome the feature you present here, but I can see that 
your idea has some caveats (some are sensible restrictions though), but the API 
doesn't give any indication of that. 

The main caveats are:

1. The implementation is limited to the first column of a layout, since it 
doesn't take the columns before the chosen column into account. Supporting 
other columns would require a bigger effort with the current approach.
2. It assumes that the layouts are all aligned on their left edge, so for 
instance if one layout has a QGroupBox as a parent, and another one has a 
QStackedWidget as a parent (they can potentially have different 
contentsMargins), the items might end up not being properly aligned afterall.
3. It seems to change the size hint of QFormLayout, but not for QGridLayout. 
This is inconsistent.

With the above caveats in mind, I am therefore wondering if it isn't a better 
API to either strictly limit it to QGroupBox+QFormLayout hierarchies or create 
a separate QWidget subclass that ensures this.

With such tighter requirement on the input I think the API would reflect much 
better the functionality it actually offers, and expose less surprises to the 
user.

Jan Arve

________________________________________
Fra: [email protected] 
[[email protected]] på vegne 
av Aurélien Gâteau [[email protected]]
Sendt: 12. juni 2014 14:40
Til: [email protected]
Emne: [Development] New class for QtWidgets: ColumnResizer

Hi,

Some time ago I created a class to help keeping columns of widgets to the
same width across layouts, so that if you have two QGroupBoxes on top of
each others with form layouts, then you can ensure the label columns of both
boxes are the same width.

I proposed it for inclusion in the KWidgetsAddons KDE framework, but it was
pointed out to me that this class could as well go to QtWidgets. You can
find the code in the KWidgetsAddons review request:
https://git.reviewboard.kde.org/r/118514/

I'd like to ask if it has a chance to be accepted? Before I start creating a
branch of QtBase for it and submit it to Gerrit.

Aurélien


_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development
_______________________________________________
Development mailing list
[email protected]
http://lists.qt-project.org/mailman/listinfo/development

Reply via email to