Skip to content

Commit

Permalink
update #877 back up required gtk to 3.18 because navbar gets lost.
Browse files Browse the repository at this point in the history
  • Loading branch information
karlkleinpaste committed May 12, 2018
1 parent a03fc0b commit 4d6c2ad
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ui/navbar_versekey.gtkbuilder
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<!-- Generated with glade 3.22.1 -->
<interface>
<requires lib="gtk+" version="3.20"/>
<requires lib="gtk+" version="3.18"/>
<object class="GtkBox" id="navbar">
<property name="visible">True</property>
<property name="can_focus">False</property>
Expand Down

3 comments on commit 4d6c2ad

@yetist
Copy link
Contributor

@yetist yetist commented on 4d6c2ad May 13, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.18 is not the lowest version, maybe 3.0?

@karlkleinpaste
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i wondered how far back to go. when i first released X4.1.0, tim dickson wrote to say he was missing the navbar. after some extensive fiddling, he discovered that version stamps like that were in the way, either that or the src/gui/utilities.h #ifdef for which UI_SUFFIX to use. the weirdness is that the navbar simply disappears, and gtk generates no error to whine about it.

i could change this to 3.0, but as a practical matter, is 3.18 or maybe 3.14 the actual earliest to be found in the wild? and might it cause any actual harm to use 3.0 instead of 3.18?

@greg-hellings
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ubuntu 14.04 LTS, which still has a year of life left on it, has GTK 3.10. Generally that's the oldest we care about. (CentOS/RHEL 6 completely lacks gtk3 from what I can tell and 7 only has it in the EPEL repository so it's pretty up to date at 3.22)

Please sign in to comment.