View Issue Details

IDProjectCategoryView StatusLast Update
0002324ardourfeaturespublic2010-02-13 04:23
Reportercolinf Assigned Topaul  
PrioritynormalSeveritytrivialReproducibilityN/A
Status assignedResolutionopen 
Product VersionSVN/2.0-ongoing 
Summary0002324: Font scaling makes Profile->set_small_screen() less necessary
DescriptionNow that Ardour's fonts are scaleable, it's quite possible for the whole GUI to fit and be workable on a 1024 x 768 display, without the hiding of UI elements that is done by Profile->set_small_screen().
Additional InformationAttached a patch to force font scaling to <= 72% on screens smaller than 1200 pixels across. I've found 72% allows Ardour to fit very nicely on my 1024 x 768 laptop, without Profile->set_small_screen().

If the screen is smaller than 1024 pixels across, it also retains the old behaviour of calling Profile->set_small_screen().

I suppose it could scale the fonts even smaller in this case, but I think it's unlikely anyone's running Ardour on anything less than 1024 x 768, and if they are they'll just have to scale them manually.

The whole lot is inside #ifndef GTKOSX, following the comment in gtk2_ardour/option_editor.cc, line 271 that says "font scaling does nothing with GDK/Quartz". I assume that's true, but there are certainly other people who'll know better than me...
TagsNo tags attached.

Activities

2008-06-23 17:58

 

scale-fonts-for-smaller-screen.patch (824 bytes)   
Index: gtk2_ardour/ardour_ui.cc
===================================================================
--- gtk2_ardour/ardour_ui.cc	(revision 3486)
+++ gtk2_ardour/ardour_ui.cc	(working copy)
@@ -3326,9 +3326,22 @@
 void
 ARDOUR_UI::setup_profile ()
 {
+#ifndef GTKOSX
 	if (gdk_screen_width() < 1200) {
+		cerr << "gdk_screen_width() = " << gdk_screen_width() <<  ", font_scale = " << (Config->get_font_scale() / 1024) << endl;
+		if (Config->get_font_scale() / 1024 > 72) {
+			Config->set_font_scale(1024 * 72);
+			cerr << "font_scale now set to " << (Config->get_font_scale() / 1024) << endl;
+		}
+		if (gdk_screen_width() < 1024) {
+			Profile->set_small_screen ();
+		}
+	}
+#else 
+	if (gdk_screen_width() < 1200) {
 		Profile->set_small_screen ();
 	}
+#endif
 
 	if (getenv ("ARDOUR_SAE")) {
 		Profile->set_sae ();

nowhiskey

2008-06-23 22:50

reporter   ~0005076

i applied the patch to rev3487 and everything looks good on my laptop with
1024 x 768. would be good if the patch comes into the source code.

cheers,
doc

naught101

2010-02-07 23:10

reporter   ~0007348

http://tracker.ardour.org/view.php?id=2950 is related.. duplicate material, even.

seablade

2010-02-13 04:23

manager   ~0007374

Wow, old patch, assigning to Paul to take a look at it and see if it is still usable.

     Seablade

Issue History

Date Modified Username Field Change
2008-06-23 17:58 colinf New Issue
2008-06-23 17:58 colinf File Added: scale-fonts-for-smaller-screen.patch
2008-06-23 22:50 nowhiskey Note Added: 0005076
2010-02-07 23:10 naught101 Note Added: 0007348
2010-02-13 04:22 seablade Status new => assigned
2010-02-13 04:22 seablade Assigned To => paul
2010-02-13 04:23 seablade Note Added: 0007374