Test fix for further osx modifier ambiguity.
[ardour.git] / gtk2_ardour / rc_option_editor.cc
index f7ba1643450f0aa380027cbd111e045cb084db79..e86c3a620ea1851b5c74e01a52bf9e41b4aa6ab6 100644 (file)
@@ -79,7 +79,7 @@ public:
                , _click_browse_button (_("Browse..."))
                , _click_emphasis_browse_button (_("Browse..."))
        {
-               Table* t = manage (new Table (3, 3));
+               Table* t = manage (new Table (4, 3));
                t->set_spacings (4);
 
                Label* l = manage (left_aligned_label (_("Use default Click:")));
@@ -88,19 +88,25 @@ public:
                _use_default_click_check_button.signal_toggled().connect (
                    sigc::mem_fun (*this, &ClickOptions::use_default_click_toggled));
 
-               l = manage (left_aligned_label (_("Click audio file:")));
+               l = manage (left_aligned_label (_("Emphasis on first beat:")));
                t->attach (*l, 0, 1, 1, 2, FILL);
-               t->attach (_click_path_entry, 1, 2, 1, 2, FILL);
+               t->attach (_use_emphasis_on_click_check_button, 1, 2, 1, 2, FILL);
+               _use_emphasis_on_click_check_button.signal_toggled().connect (
+                   sigc::mem_fun (*this, &ClickOptions::use_emphasis_on_click_toggled));
+
+               l = manage (left_aligned_label (_("Click audio file:")));
+               t->attach (*l, 0, 1, 2, 3, FILL);
+               t->attach (_click_path_entry, 1, 2, 2, 3, FILL);
                _click_browse_button.signal_clicked ().connect (
                    sigc::mem_fun (*this, &ClickOptions::click_browse_clicked));
-               t->attach (_click_browse_button, 2, 3, 1, 2, FILL);
+               t->attach (_click_browse_button, 2, 3, 2, 3, FILL);
 
                l = manage (left_aligned_label (_("Click emphasis audio file:")));
-               t->attach (*l, 0, 1, 2, 3, FILL);
-               t->attach (_click_emphasis_path_entry, 1, 2, 2, 3, FILL);
+               t->attach (*l, 0, 1, 3, 4, FILL);
+               t->attach (_click_emphasis_path_entry, 1, 2, 3, 4, FILL);
                _click_emphasis_browse_button.signal_clicked ().connect (
                    sigc::mem_fun (*this, &ClickOptions::click_emphasis_browse_clicked));
-               t->attach (_click_emphasis_browse_button, 2, 3, 2, 3, FILL);
+               t->attach (_click_emphasis_browse_button, 2, 3, 3, 4, FILL);
 
                _box->pack_start (*t, false, false);
 
@@ -110,9 +116,11 @@ public:
                if (_rc_config->get_click_sound ().empty() &&
                    _rc_config->get_click_emphasis_sound().empty()) {
                        _use_default_click_check_button.set_active (true);
+                       _use_emphasis_on_click_check_button.set_active (true);
 
                } else {
                        _use_default_click_check_button.set_active (false);
+                       _use_emphasis_on_click_check_button.set_active (false);
                }
        }
 
@@ -122,6 +130,9 @@ public:
                        _click_path_entry.set_text (_rc_config->get_click_sound());
                } else if (p == "click-emphasis-sound") {
                        _click_emphasis_path_entry.set_text (_rc_config->get_click_emphasis_sound());
+               } else if (p == "use-click-emphasis") {
+                       bool x = _rc_config->get_use_click_emphasis ();
+                       _use_emphasis_on_click_check_button.set_active (x);
                }
        }
 
@@ -129,6 +140,7 @@ public:
        {
                parameter_changed ("click-sound");
                parameter_changed ("click-emphasis-sound");
+               parameter_changed ("use-click-emphasis");
        }
 
 private:
@@ -196,8 +208,18 @@ private:
                }
        }
 
+       void use_emphasis_on_click_toggled ()
+       {
+               if (_use_emphasis_on_click_check_button.get_active ()) {
+                       _rc_config->set_use_click_emphasis(true);
+               } else {
+                       _rc_config->set_use_click_emphasis(false);
+               }
+       }
+
        RCConfiguration* _rc_config;
        CheckButton _use_default_click_check_button;
+       CheckButton _use_emphasis_on_click_check_button;
        Entry _click_path_entry;
        Entry _click_emphasis_path_entry;
        Button _click_browse_button;
@@ -308,14 +330,16 @@ static const struct {
           Option/Alt = Mod1
        */
        { "Key|Shift", GDK_SHIFT_MASK },
-       { "Command", GDK_META_MASK },
+       { "Command", GDK_MOD2_MASK },
        { "Control", GDK_CONTROL_MASK },
        { "Option", GDK_MOD1_MASK },
-       { "Command-Shift", GDK_META_MASK|GDK_SHIFT_MASK },
-       { "Command-Option", GDK_MOD1_MASK|GDK_META_MASK },
+       { "Command-Shift", GDK_MOD2_MASK|GDK_SHIFT_MASK },
+       { "Command-Option", GDK_MOD2_MASK|GDK_MOD1_MASK },
+       { "Command-Option-Control", GDK_MOD2_MASK|GDK_MOD1_MASK|GDK_CONTROL_MASK },
+       { "Option-Control", GDK_MOD1_MASK|GDK_CONTROL_MASK },
        { "Option-Shift", GDK_MOD1_MASK|GDK_SHIFT_MASK },
        { "Control-Shift", GDK_CONTROL_MASK|GDK_SHIFT_MASK },
-       { "Shift-Command-Option", GDK_MOD5_MASK|GDK_SHIFT_MASK|GDK_META_MASK },
+       { "Shift-Command-Option", GDK_MOD5_MASK|GDK_SHIFT_MASK|GDK_MOD2_MASK },
 
 #else
        { "Key|Shift", GDK_SHIFT_MASK },
@@ -323,9 +347,10 @@ static const struct {
        { "Alt", GDK_MOD1_MASK },
        { "Control-Shift", GDK_CONTROL_MASK|GDK_SHIFT_MASK },
        { "Control-Alt", GDK_CONTROL_MASK|GDK_MOD1_MASK },
-       { "Shift-Alt", GDK_SHIFT_MASK|GDK_MOD1_MASK },
        { "Control-Shift-Alt", GDK_CONTROL_MASK|GDK_SHIFT_MASK|GDK_MOD1_MASK },
        { "Alt-Windows", GDK_MOD1_MASK|GDK_MOD4_MASK },
+       { "Alt-Shift", GDK_MOD1_MASK|GDK_SHIFT_MASK },
+       { "Alt-Shift-Windows", GDK_MOD1_MASK|GDK_SHIFT_MASK|GDK_MOD4_MASK },
        { "Mod2", GDK_MOD2_MASK },
        { "Mod3", GDK_MOD3_MASK },
        { "Windows", GDK_MOD4_MASK },
@@ -346,6 +371,7 @@ public:
                  _insert_note_button_adjustment (3, 1, 5),
                  _insert_note_button_spin (_insert_note_button_adjustment)
        {
+               const Glib::ustring restart_msg = _("\nChanges to this setting will only persist after your project has been saved.");
                /* internationalize and prepare for use with combos */
 
                vector<string> dumb;
@@ -355,7 +381,8 @@ public:
 
                set_popdown_strings (_edit_modifier_combo, dumb);
                _edit_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::edit_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_edit_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1 + button 3</b>%2"),  Keyboard::copy_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == Keyboard::edit_modifier ()) {
                                _edit_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -416,7 +443,8 @@ public:
 
                set_popdown_strings (_delete_modifier_combo, dumb);
                _delete_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::delete_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_delete_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1 + button 3</b>%2"), Keyboard::tertiary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == Keyboard::delete_modifier ()) {
                                _delete_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -445,7 +473,8 @@ public:
 
                set_popdown_strings (_insert_note_modifier_combo, dumb);
                _insert_note_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::insert_note_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_insert_note_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1 + button 1</b>%2"), Keyboard::copy_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == Keyboard::insert_note_modifier ()) {
                                _insert_note_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -481,7 +510,8 @@ public:
                /* copy modifier */
                set_popdown_strings (_copy_modifier_combo, dumb);
                _copy_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::copy_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_copy_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::copy_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) Keyboard::CopyModifier) {
                                _copy_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -501,7 +531,8 @@ public:
                /* constraint modifier */
                set_popdown_strings (_constraint_modifier_combo, dumb);
                _constraint_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::constraint_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_constraint_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::secondary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::constraint_modifier ()) {
                                _constraint_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -527,7 +558,8 @@ public:
                /* trim_contents */
                set_popdown_strings (_trim_contents_combo, dumb);
                _trim_contents_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::trim_contents_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_trim_contents_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::primary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::trim_contents_modifier ()) {
                                _trim_contents_combo.set_active_text (S_(modifiers[x].name));
@@ -547,7 +579,8 @@ public:
                /* anchored trim */
                set_popdown_strings (_trim_anchored_combo, dumb);
                _trim_anchored_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::trim_anchored_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_trim_anchored_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::tertiary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::trim_anchored_modifier ()) {
                                _trim_anchored_combo.set_active_text (S_(modifiers[x].name));
@@ -590,7 +623,8 @@ public:
                /* note resize relative */
                set_popdown_strings (_note_size_relative_combo, dumb);
                _note_size_relative_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::note_size_relative_modifier_chosen));
-
+               Gtkmm2ext::UI::instance()->set_tip (_note_size_relative_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::primary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::note_size_relative_modifier ()) {
                                _note_size_relative_combo.set_active_text (S_(modifiers[x].name));
@@ -617,7 +651,13 @@ public:
                /* ignore snap */
                set_popdown_strings (_snap_modifier_combo, dumb);
                _snap_modifier_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::snap_modifier_chosen));
-
+#ifdef GTKOSX
+               Glib::ustring mod_str = string_compose (X_("%1-%2"), Keyboard::level4_modifier_name (), Keyboard::tertiary_modifier_name ());
+#else
+               Glib::ustring mod_str = Keyboard::secondary_modifier_name();
+#endif
+               Gtkmm2ext::UI::instance()->set_tip (_snap_modifier_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), mod_str, restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) Keyboard::snap_modifier ()) {
                                _snap_modifier_combo.set_active_text (S_(modifiers[x].name));
@@ -637,7 +677,13 @@ public:
                /* snap delta */
                set_popdown_strings (_snap_delta_combo, dumb);
                _snap_delta_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::snap_delta_modifier_chosen));
-
+#ifdef GTKOSX
+               mod_str = Keyboard::level4_modifier_name ();
+#else
+               mod_str = string_compose (X_("%1-%2"), Keyboard::secondary_modifier_name (), Keyboard::level4_modifier_name ());
+#endif
+               Gtkmm2ext::UI::instance()->set_tip (_snap_delta_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), mod_str, restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) Keyboard::snap_delta_modifier ()) {
                                _snap_delta_combo.set_active_text (S_(modifiers[x].name));
@@ -664,6 +710,8 @@ public:
                set_popdown_strings (_trim_overlap_combo, dumb);
                _trim_overlap_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::trim_overlap_modifier_chosen));
 
+               Gtkmm2ext::UI::instance()->set_tip (_trim_overlap_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::tertiary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::trim_overlap_modifier ()) {
                                _trim_overlap_combo.set_active_text (S_(modifiers[x].name));
@@ -690,6 +738,9 @@ public:
                set_popdown_strings (_fine_adjust_combo, dumb);
                _fine_adjust_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::fine_adjust_modifier_chosen));
 
+               mod_str = string_compose (X_("%1-%2"), Keyboard::secondary_modifier_name (), Keyboard::tertiary_modifier_name ());
+               Gtkmm2ext::UI::instance()->set_tip (_fine_adjust_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), mod_str, restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::fine_adjust_modifier ()) {
                                _fine_adjust_combo.set_active_text (S_(modifiers[x].name));
@@ -710,6 +761,8 @@ public:
                set_popdown_strings (_push_points_combo, dumb);
                _push_points_combo.signal_changed().connect (sigc::mem_fun(*this, &KeyboardOptions::push_points_modifier_chosen));
 
+               Gtkmm2ext::UI::instance()->set_tip (_push_points_combo,
+                                                   (string_compose (_("<b>Recommended Setting: %1</b>%2"), Keyboard::primary_modifier_name (), restart_msg)));
                for (int x = 0; modifiers[x].name; ++x) {
                        if (modifiers[x].modifier == (guint) ArdourKeyboard::push_points_modifier ()) {
                                _push_points_combo.set_active_text (S_(modifiers[x].name));
@@ -1843,7 +1896,7 @@ RCOptionEditor::RCOptionEditor ()
        add_option (_("Misc"),
             new BoolOption (
                     "periodic-safety-backups",
-                    _("Automatically save the session file at regular intervals"),
+                    _("Make periodic backups of the session file"),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::get_periodic_safety_backups),
                     sigc::mem_fun (*_rc_config, &RCConfiguration::set_periodic_safety_backups)
                     ));