Opened 14 years ago

Closed 12 years ago

Last modified 8 years ago

#1349 closed bug (Fixed)

deluge-gtk side panel drawn incorrectly when using specific GTK+ themes

Reported by: lexa2 Owned by:
Priority: trivial Milestone:
Component: GTK UI Version: 1.3.1
Keywords: gui side panel Cc: bertrand.croq+deluge@gmail.com

Description

deluge-gtk draw side panel as GtkTreeView inside GtkScrolledWindow.
To make items like "All", "Seeding", "Active" not indented too far to the left it sets GtkTreeView->level-indentation to -35 which violates GTK+ requirements (as described at http://library.gnome.org/devel/gtk/stable/GtkTreeView.html#GtkTreeView--level-indentation) and makes side panel prone to the incorrect appearance when GtkTreeView's expander-size style is set to anything lower than default 12 (and generally speaking even with expander-size=12 sidebar looks like being drawn wrong with so-called "compact" themes like "Clearlooks Compact" or "Human Compact").

Graphical "glitch" is that treeview items got placed too far to the left making them crop with the right border of the treeeview (look at the attached screenshot).

Thus GUI results in being drawn wrong when using some perfectly legal gnome themes due to deluge not following GTK+ docs so I consider this to be a bug that require some modifications to the way deluge use to draw side panel.

Attachments (2)

2010-08-25_Deluge_1.2.3_bug_N01.png (11.4 KB ) - added by lexa2 14 years ago.
Re-attaching with different name due to Trac not able to handle names with # character correctly.
deluge.png (48.9 KB ) - added by bcroq 14 years ago.
screenshot of 1.3.1-1ubuntu1 with the "Xfce" theme

Download all attachments as: .zip

Change History (11)

by lexa2, 14 years ago

Re-attaching with different name due to Trac not able to handle names with # character correctly.

comment:1 by andar, 14 years ago

If you can suggest a way on how we can get the current theme's indentation level, then that'd be appreciated. As far as I know, there is no real way to do so and thus we've been stuck using -35 to render the sidebar as we want it.

comment:2 by John Garland, 14 years ago

Milestone: Future1.4.0

comment:3 by andar, 14 years ago

Milestone: 1.4.01.3.1
Resolution: fixed
Status: newclosed

Fixed in 1.3-stable.

comment:4 by bcroq, 14 years ago

Cc: bertrand.croq+deluge@gmail.com added
Resolution: fixed
Status: closedreopened

I still get the problem using deluge-1.3.1-1ubuntu1 on xubuntu natty, whatever the theme I try (I currently use the "Xfce" theme)

by bcroq, 14 years ago

Attachment: deluge.png added

screenshot of 1.3.1-1ubuntu1 with the "Xfce" theme

comment:5 by Calum, 14 years ago

Component: othergtkui
Milestone: 1.3.11.3.2
Priority: minortrivial
Type: defectbug
Version: 1.2.31.3.1

comment:6 by Calum, 13 years ago

Milestone: 1.3.21.3.x

comment:7 by Calum, 13 years ago

Not sure how this can be fixed for XFCE theme but you could edit either of the following two values in filtertreeview.py to see what works best.

self.label_view.set_level_indentation = -35 
GtkTreeView::expander-size = 15

For reference this is andar's workaround: 2a3eb0

comment:8 by Calum, 12 years ago

Resolution: fixed
Status: reopenedclosed

Testing with Xubuntu 12.04 showed no issues so closing.

comment:9 by Calum, 8 years ago

Milestone: 1.3.x

Milestone deleted

Note: See TracTickets for help on using tickets.