Torrent addition dialog

Bug #178167 reported by attilaolah
2
Affects Status Importance Assigned to Milestone
qBittorrent
Fix Released
Undecided
Christophe Dumez

Bug Description

I just think you should use
    setResizeMode(0, QtGui.QHeaderView.Stretch)
instead of
    setResizeMode(2, QtGui.QHeaderView.Stretch)
or
    setStretchLastSection(1)
so when I resize the dialog box, I don't need to resize the tree view, too. The other columns should be set with resizeColumnToContents(index).

Just my opinion.
aatiis

Revision history for this message
attilaolah (attilaolah) wrote :

btw why not make that tree view sortable?

Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

Fixed in trunk. Thanks.

You're a pyQt4 programmer I'm guessing ;)

Changed in qbittorrent:
assignee: nobody → hydr0g3n
status: New → Fix Committed
milestone: none → 1.1.0
Changed in qbittorrent:
status: Fix Committed → Fix Released
Revision history for this message
attilaolah (attilaolah) wrote :

In file src/torrentAddition.h:

Why is the first column not stretched, but the last one?
line 82:
      torrentContentList->header()->resizeSection(0, 200);
      //torrentContentList->header()->setResizeMode(0, QHeaderView::Stretch);

I think you should let the first column stretch (after resizing it to 200), and additionally, prevent the last one from straching, using:
line82+
      torrentContentList->header()->setStretchLastSection(0);

Or additionally, you could resize the last two columns to fit the content, and then make the first column stretch. Or you could just leave those two columns fixed.

By the way, I'd still like to be able to sort that tree view, i.e. to give smaller files a higher priority.

And it would be nice be able to expand/collapse subdirectiries, not just the root directory.

Nice wishlist, I suppose :)

Changed in qbittorrent:
status: Fix Released → New
Revision history for this message
attilaolah (attilaolah) wrote :

Also, I have some complains about the GUI... I think that rootIsDecorated should be set to 0, and allColumnsShowFocus should be set to true on both upload and download tabs, since they're not expandable.

BTW, is this the right place to discuss this?

Revision history for this message
attilaolah (attilaolah) wrote :

Also since the connection icon is now down in the status bar, I don't think we need that last separator in the toolbar.

You probably have better things to to do than to constantly update the GUI, so I could provide you diffs if you approve the changes.

Sorry for the dozens of bug reports. This will be the last one for tonight. I'll go to sleep now :)

Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

aatis > I fixed all reported problem and v1.3.4 should be ok. Thanks for the reports.

Changed in qbittorrent:
milestone: 1.1.0 → 1.3.4
status: New → Fix Committed
Revision history for this message
Christophe Dumez (hydr0g3n) wrote :

v1.3.4 was released.

Changed in qbittorrent:
status: Fix Committed → Fix Released
Revision history for this message
attilaolah (attilaolah) wrote :

Thanks, I'll check it out. Keep up the good work, Chris.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.