fix #4663, partly by more efficiently generating the relevant string, and secondly...
[ardour.git] / gtk2_ardour / editor_regions.cc
index ae9c48cc3a871f8af6752ff59604cb866845fd20..672a0c5b8edafe38b4ce7591195323b2ce818d3f 100644 (file)
@@ -59,6 +59,12 @@ using namespace Glib;
 using namespace Editing;
 using Gtkmm2ext::Keyboard;
 
+struct ColumnInfo {
+    int         index;
+    const char* label;
+    const char* tooltip;
+};
+
 EditorRegions::EditorRegions (Editor* e)
        : EditorComponent (e)
        , old_focus (0)
@@ -72,7 +78,6 @@ EditorRegions::EditorRegions (Editor* e)
        , expanded (false)
 {
        _display.set_size_request (100, -1);
-       _display.set_name ("RegionListDisplay");
        _display.set_rules_hint (true);
 
        /* Try to prevent single mouse presses from initiating edits.
@@ -101,66 +106,36 @@ EditorRegions::EditorRegions (Editor* e)
        TreeViewColumn* col;
        Gtk::Label* l;
 
-       col = _display.get_column (0);
-       l = manage (new Label (_("Regions")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region name, with number of channels in []'s"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (1);
-       l = manage (new Label(_("Position")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Length of the region"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (2);
-       l = manage (new Label(_("End")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Length of the region"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (3);
-       l = manage (new Label(_("End")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Length of the region"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (4);
-       l = manage (new Label(_("Sync")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Position of region sync point, relative to start of the region"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (5);
-       l = manage (new Label(_("Sync")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region fade-in enabled?"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (6);
-       l = manage (new Label(_("Fade In")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region fade-out enabled?"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (7);
-       l = manage (new Label(_("L")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region position locked?"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (8);
-       l = manage (new Label(_("G")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region position glued to Bars|Beats time?"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (9);
-       l = manage (new Label(_("M")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region muted?"));
-       col->set_widget (*l);
-       l->show ();
-       col = _display.get_column (10);
-       l = manage (new Label(_("O")));
-       ARDOUR_UI::instance()->set_tip (*l, _("Region opaque (blocks regions below it from being heard)?"));
-       col->set_widget (*l);
-       l->show ();
-
-       // _display.append_column (_("Used"), _columns.used);
-       // _display.append_column (_("Path"), _columns.path);
+       ColumnInfo ci[] = {
+               { 0, _("Region"), _("Region name, with number of channels in []'s") },
+               { 1, _("Position"),  _("Position of start of region") },
+               { 2, _("End"),  _("Position of end of region") },
+               { 3, _("Length"),  _("Length of the region") },
+               { 4, _("Sync"),  _("Position of region sync point, relative to start of the region") },
+               { 5, _("Fade In"),  _("Length of region fade-in (units: secondary clock), () if disabled") },
+               { 6, _("Fade Out"),  _("Length of region fade-out (units: secondary clock), () if dsisabled") },
+               { 7, _("L"),  _("Region position locked?") },
+               { 8, _("G"),  _("Region position glued to Bars|Beats time?") },
+               { 9, _("M"),  _("Region muted?") },
+               { 10, _("O"),  _("Region opaque (blocks regions below it from being heard)?") },
+               { -1, 0, 0 }
+       };
+       
+       for (int i = 0; ci[i].index >= 0; ++i) {
+               col = _display.get_column (ci[i].index);
+               l = manage (new Label (ci[i].label));
+               ARDOUR_UI::instance()->set_tip (*l, ci[i].tooltip);
+               col->set_widget (*l);
+               l->show ();
+
+               if (ci[i].index > 6) {
+                       col->set_expand (false);
+                       col->set_alignment (ALIGN_CENTER);
+               } 
+       }
+
        _display.set_headers_visible (true);
-       //_display.set_grid_lines (TREE_VIEW_GRID_LINES_BOTH);
+       _display.set_rules_hint ();
 
        /* show path as the row tooltip */
        _display.set_tooltip_column (14); /* path */
@@ -173,10 +148,10 @@ EditorRegions::EditorRegions (Editor* e)
        _display.get_selection()->set_select_function (sigc::mem_fun (*this, &EditorRegions::selection_filter));
 
        TreeViewColumn* tv_col = _display.get_column(0);
-       tv_col->set_resizable (true);
        CellRendererText* renderer = dynamic_cast<CellRendererText*>(_display.get_column_cell_renderer (0));
        tv_col->add_attribute(renderer->property_text(), _columns.name);
        tv_col->add_attribute(renderer->property_foreground_gdk(), _columns.color_);
+       tv_col->set_expand (true);
 
        CellRendererToggle* locked_cell = dynamic_cast<CellRendererToggle*> (_display.get_column_cell_renderer (7));
        locked_cell->property_activatable() = true;
@@ -209,17 +184,6 @@ EditorRegions::EditorRegions (Editor* e)
        _display.get_selection()->set_mode (SELECTION_MULTIPLE);
        _display.add_object_drag (_columns.region.index(), "regions");
 
-
-       /* only the first column (Region name) is resizable */
-
-       for (int i = 1; ; ++i) {
-               TreeViewColumn* col = _display.get_column (i);
-               if (!col) {
-                       break;
-               } 
-               col->set_resizable (false);
-       }
-
        /* setup DnD handling */
 
        list<TargetEntry> region_list_target_table;
@@ -521,6 +485,8 @@ EditorRegions::region_changed (boost::shared_ptr<Region> r, const PropertyChange
        our_interests.add (ARDOUR::Properties::opaque);
        our_interests.add (ARDOUR::Properties::fade_in);
        our_interests.add (ARDOUR::Properties::fade_out);
+       our_interests.add (ARDOUR::Properties::fade_in_active);
+       our_interests.add (ARDOUR::Properties::fade_out_active);
 
        if (what_changed.contains (our_interests)) {
 
@@ -722,7 +688,7 @@ EditorRegions::update_all_rows ()
 }
 
 void
-EditorRegions::format_position (framepos_t pos, char* buf, size_t bufsize)
+EditorRegions::format_position (framepos_t pos, char* buf, size_t bufsize, bool onoff)
 {
        Timecode::BBT_Time bbt;
        Timecode::Time timecode;
@@ -730,7 +696,11 @@ EditorRegions::format_position (framepos_t pos, char* buf, size_t bufsize)
        switch (ARDOUR_UI::instance()->secondary_clock->mode ()) {
        case AudioClock::BBT:
                _session->tempo_map().bbt_time (pos, bbt);
-               snprintf (buf, bufsize, "%03d|%02d|%04d" , bbt.bars, bbt.beats, bbt.ticks);
+               if (onoff) {
+                       snprintf (buf, bufsize, "%03d|%02d|%04d" , bbt.bars, bbt.beats, bbt.ticks);
+               } else {
+                       snprintf (buf, bufsize, "(%03d|%02d|%04d)" , bbt.bars, bbt.beats, bbt.ticks);
+               }
                break;
 
        case AudioClock::MinSec:
@@ -745,17 +715,29 @@ EditorRegions::format_position (framepos_t pos, char* buf, size_t bufsize)
                mins = (int) floor (left / (_session->frame_rate() * 60.0f));
                left -= (framecnt_t) floor (mins * _session->frame_rate() * 60.0f);
                secs = left / (float) _session->frame_rate();
-               snprintf (buf, bufsize, "%02d:%02d:%06.3f", hrs, mins, secs);
+               if (onoff) {
+                       snprintf (buf, bufsize, "%02d:%02d:%06.3f", hrs, mins, secs);
+               } else {
+                       snprintf (buf, bufsize, "(%02d:%02d:%06.3f)", hrs, mins, secs);
+               }
                break;
 
        case AudioClock::Frames:
-               snprintf (buf, bufsize, "%" PRId64, pos);
+               if (onoff) {
+                       snprintf (buf, bufsize, "%" PRId64, pos);
+               } else {
+                       snprintf (buf, bufsize, "(%" PRId64 ")", pos);
+               }
                break;
 
        case AudioClock::Timecode:
        default:
                _session->timecode_time (pos, timecode);
-               snprintf (buf, bufsize, "%02d:%02d:%02d:%02d", timecode.hours, timecode.minutes, timecode.seconds, timecode.frames);
+               if (onoff) {
+                       snprintf (buf, bufsize, "%02d:%02d:%02d:%02d", timecode.hours, timecode.minutes, timecode.seconds, timecode.frames);
+               } else {
+                       snprintf (buf, bufsize, "(%02d:%02d:%02d:%02d)", timecode.hours, timecode.minutes, timecode.seconds, timecode.frames);
+               }
                break;
        }
 }
@@ -887,16 +869,9 @@ EditorRegions::populate_row_fade_in (boost::shared_ptr<Region> region, TreeModel
                if (used > 1) {
                        row[_columns.fadein] = _("Multiple");
                } else {
-
-                       char buf[16];
-                       format_position (audioregion->fade_in()->back()->when, buf, sizeof (buf));
+                       char buf[32];
+                       format_position (audioregion->fade_in()->back()->when, buf, sizeof (buf), audioregion->fade_in_active());
                        row[_columns.fadein] = buf;
-
-                       if (audioregion->fade_in_active()) {
-                               row[_columns.fadein] = string_compose("%1%2%3", " ", buf, " ");
-                       } else {
-                               row[_columns.fadein] = string_compose("%1%2%3", "(", buf, ")");
-                       }
                }
        }
 }
@@ -910,14 +885,9 @@ EditorRegions::populate_row_fade_out (boost::shared_ptr<Region> region, TreeMode
                if (used > 1) {
                        row[_columns.fadeout] = _("Multiple");
                } else {
-                       char buf[16];
-                       format_position (audioregion->fade_out()->back()->when, buf, sizeof (buf));
-
-                       if (audioregion->fade_out_active()) {
-                               row[_columns.fadeout] = string_compose("%1%2%3", " ", buf, " ");
-                       } else {
-                               row[_columns.fadeout] = string_compose("%1%2%3", "(", buf, ")");
-                       }
+                       char buf[32];
+                       format_position (audioregion->fade_out()->back()->when, buf, sizeof (buf), audioregion->fade_out_active());
+                       row[_columns.fadeout] = buf;
                }
        }
 }