Should not show tab in Properties window, when there is only one tab

[Andreas Nilsson]

Binary package hint: banshee

(real bug is https://bugzilla.gnome.org/show_bug.cgi?id=615797 but filing it here too so we can track it as a papercut)

The track properties dialog sports a tabbed interface, but there is only one tab, so that looks odd and it makes it not really a tabbed interface. It also looks a bit odd.

This is probably happening because it's pretty much shares everything with the Edit Track dialog, including this tab.

We probably should special case a single tab.

[Vish (vish)]

Screenshot of the problem : http://bugzilla-attachments.gnome.org/attachment.cgi?id=158767

[Zachary (zachary418)]

Since the Track Editor contains the Track Properties tab anyway, this window is redundant. It would be more compact to remove this separate window altogether and rename Edit Track Information to something like Track Info.

[Buttink (buttink)]

This is all that should be needed for the simple fix

git diff
diff --git a/src/Core/Banshee.ThickClient/Banshee.Gui.TrackEditor/TrackEditorDia
index 949489b..8290213 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Gui.TrackEditor/TrackEditorDialog.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Gui.TrackEditor/TrackEditorDialog.cs
@@ -257,6 +257,10 @@ namespace Banshee.Gui.TrackEditor
             if (page_to_focus != null) {
                 notebook.CurrentPage = notebook.PageNum (page_to_focus);
             }
+
+ if (pages.Count < 2) {
+ notebook.ShowTabs = false;
+ }
         }

         private void BuildFooter ()
diff --git a/src/Hyena b/src/Hyena
--- a/src/Hyena
+++ b/src/Hyena
@@ -1 +1 @@
-Subproject commit 507573f2b0b1a462df491bb236bba9a3e86e2869
+Subproject commit 507573f2b0b1a462df491bb236bba9a3e86e2869-dirty

I'd do it myself, but I'm having problems with autoconf and can't run it to test it. Another solution would be to have the same Dialog for both, but just make "Properties" start on the "Properties" tab.

[Vish (vish)]

@Buttink , Can you submit that patch on : https://bugzilla.gnome.org/show_bug.cgi?id=615797 ? The Banshee developers can comment on the patch.

[William Friesen (william-friesen)]

I've tested the above patch, and it seems that pages.Count is not a direct indicator of the pages that will be shown. When I ran it, pages.Count gave the value 2, when there was only 1 tab showing. Also, when viewing the "Edit Track Information" dialog, pages.Count was 5, when only 4 tabs were showing.

[Chow Loong Jin (hyperair)]

A crude method of counting the visible would be to declare an integer, and increment it every time page.Widget.Show() is called in the BuildNotebook() function.

[Chow Loong Jin (hyperair)]

Er I mean visible pages.

[Buttink (buttink)]

@Vish will do. But first, I'm going to try and find a way to easily find the number of visible pages. Also, get autoconf to work so I can test it. :)

[William Friesen (william-friesen)]

For me, the extra tab that was hiding came from the Podcasts extension. When added to the notebook, this had the property Order = 40, which duplicated the Order value of a previous page in the notebook. I have submitted a patch to the banshee bug that determines the amount of visible pages by counting the unique page.Order values, and calls notebook.ShowTabs when there is only one visible page.

[Vish (vish)]

Buttink, if you are having any trouble or have any questions, send a mail to Papercuts Ninja team: <https://launchpad.net/~papercuts-ninja> .

[Vish (vish)]

Nice work William Friesen !

[Vish (vish)]

@Buttink, Seems William got to the bug, no worries, banshee has other papercuts too.

Buttink/ William , feel free to knock off any of the other banshee bugs ;-) > https://launchpad.net/hundredpapercuts/+milestone/nt1-music .