Should not show tab in Properties window, when there is only one tab
Bug Description
Binary package hint: banshee
(real bug is https:/
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.
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.
This is all that should be needed for the simple fix
git diff
diff --git a/src/Core/
index 949489b..8290213 100644
--- a/src/Core/
+++ b/src/Core/
@@ -257,6 +257,10 @@ namespace Banshee.
if (page_to_focus != null) {
}
+
+ 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 507573f2b0b1a46
+Subproject commit 507573f2b0b1a46
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.
@Buttink , Can you submit that patch on : https:/
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.
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.
Er I mean visible pages.
@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. :)
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.
Buttink, if you are having any trouble or have any questions, send a mail to Papercuts Ninja team: <https:/
Nice work William Friesen !
@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:/
Screenshot of the problem : http://
bugzilla- attachments. gnome.org/ attachment. cgi?id= 158767