more miscellaneous fixes to make things even nicer

git-svn-id: svn://localhost/trunk/ardour2@153 d708f5d6-7413-0410-9779-e7cbd77b26cf
This commit is contained in:
Paul Davis 2005-11-30 19:34:09 +00:00
parent 327018191b
commit 53d337a168
6 changed files with 134 additions and 103 deletions

View file

@ -79,19 +79,20 @@ Editor::Cursor::set_position (jack_nframes_t frame)
current_frame = frame; current_frame = frame;
if (new_pos == points.front().get_x()) { if (new_pos != points.front().get_x()) {
/* change in position is not visible, so just raise it */
canvas_item.raise_to_top();
return;
}
points.front().set_x (new_pos); points.front().set_x (new_pos);
points.back().set_x (new_pos); points.back().set_x (new_pos);
// cerr << "set cursor2 al points, nc = " << points->num_points << endl; cerr << "new cursor points = "
<< points.front().get_x() << ',' << points.front().get_y()
<< " .. "
<< points.back().get_x() << ',' << points.back().get_y()
<< endl;
canvas_item.property_points() = points; canvas_item.property_points() = points;
}
canvas_item.raise_to_top(); canvas_item.raise_to_top();
} }
@ -99,8 +100,7 @@ void
Editor::Cursor::set_length (double units) Editor::Cursor::set_length (double units)
{ {
length = units; length = units;
points.back().set_x (points.front().get_y() + length); points.back().set_y (points.front().get_y() + length);
// cerr << "set cursor3 al points, nc = " << points->num_points << endl;
canvas_item.property_points() = points; canvas_item.property_points() = points;
} }
@ -108,7 +108,6 @@ void
Editor::Cursor::set_y_axis (double position) Editor::Cursor::set_y_axis (double position)
{ {
points.front().set_y (position); points.front().set_y (position);
points.back().set_x (position + length); points.back().set_y (position + length);
// cerr << "set cursor4 al points, nc = " << points->num_points << endl;
canvas_item.property_points() = points; canvas_item.property_points() = points;
} }

View file

@ -122,11 +122,6 @@ ExportDialog::ExportDialog(PublicEditor& e, AudioRegion* r)
set_name ("ExportWindow"); set_name ("ExportWindow");
add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK); add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
add (vpacker);
vpacker.set_border_width (10);
vpacker.set_spacing (10);
file_selector = 0; file_selector = 0;
spec.running = false; spec.running = false;
@ -172,7 +167,7 @@ ExportDialog::ExportDialog(PublicEditor& e, AudioRegion* r)
track_scroll.set_policy(Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC); track_scroll.set_policy(Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC);
master_scroll.set_policy(Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC); master_scroll.set_policy(Gtk::POLICY_AUTOMATIC, Gtk::POLICY_AUTOMATIC);
vpacker.pack_start (file_frame, false, false); get_vbox()->pack_start (file_frame, false, false);
hpacker.set_spacing (5); hpacker.set_spacing (5);
hpacker.set_border_width (5); hpacker.set_border_width (5);
@ -196,13 +191,13 @@ ExportDialog::ExportDialog(PublicEditor& e, AudioRegion* r)
hpacker.pack_start (track_vpacker); hpacker.pack_start (track_vpacker);
} }
vpacker.pack_start (hpacker); get_vbox()->pack_start (hpacker);
track_selector_button.set_name ("EditorGTKButton"); track_selector_button.set_name ("EditorGTKButton");
track_selector_button.signal_clicked().connect (mem_fun(*this, &ExportDialog::track_selector_button_click)); track_selector_button.signal_clicked().connect (mem_fun(*this, &ExportDialog::track_selector_button_click));
vpacker.pack_start (button_box, false, false); get_vbox()->pack_start (button_box, false, false);
vpacker.pack_start (progress_bar, false, false); get_vbox()->pack_start (progress_bar, false, false);
Gtkmm2ext::set_size_request_to_display_given_text (file_entry, X_("Kg/quite/a/reasonable/size/for/files/i/think"), 5, 8); Gtkmm2ext::set_size_request_to_display_given_text (file_entry, X_("Kg/quite/a/reasonable/size/for/files/i/think"), 5, 8);

View file

@ -73,7 +73,6 @@ class ExportDialog : public ArdourDialog
PublicEditor& editor; PublicEditor& editor;
ARDOUR::Session* session; ARDOUR::Session* session;
ARDOUR::AudioRegion* audio_region; ARDOUR::AudioRegion* audio_region;
Gtk::VBox vpacker;
Gtk::VBox track_vpacker; Gtk::VBox track_vpacker;
Gtk::HBox hpacker; Gtk::HBox hpacker;
Gtk::HBox button_box; Gtk::HBox button_box;

View file

@ -102,7 +102,7 @@ PlaylistSelector::show_for (RouteUI* ruix)
clear_map (); clear_map ();
select_connection.disconnect (); select_connection.disconnect ();
model.clear (); model->clear ();
session->foreach_playlist (this, &PlaylistSelector::add_playlist_to_map); session->foreach_playlist (this, &PlaylistSelector::add_playlist_to_map);

View file

@ -40,6 +40,8 @@
#include "i18n.h" #include "i18n.h"
using namespace Gtkmm2ext; using namespace Gtkmm2ext;
using namespace Gtk;
using namespace Glib;
using std::map; using std::map;
pthread_t UI::gui_thread; pthread_t UI::gui_thread;
@ -48,8 +50,8 @@ UI *UI::theGtkUI = 0;
UI::UI (string name, int *argc, char ***argv, string rcfile) UI::UI (string name, int *argc, char ***argv, string rcfile)
: _ui_name (name) : _ui_name (name)
{ {
theMain = new Gtk::Main (argc, argv); theMain = new Main (argc, argv);
tips = new Gtk::Tooltips; tips = new Tooltips;
if (pthread_key_create (&thread_request_buffer_key, 0)) { if (pthread_key_create (&thread_request_buffer_key, 0)) {
cerr << _("cannot create thread request buffer key") << endl; cerr << _("cannot create thread request buffer key") << endl;
@ -73,8 +75,6 @@ UI::UI (string name, int *argc, char ***argv, string rcfile)
return; return;
} }
load_rcfile (rcfile);
errors = new TextViewer (850,100); errors = new TextViewer (850,100);
errors->text().set_editable (false); errors->text().set_editable (false);
errors->text().set_name ("ErrorText"); errors->text().set_name ("ErrorText");
@ -85,10 +85,12 @@ UI::UI (string name, int *argc, char ***argv, string rcfile)
errors->set_title (title); errors->set_title (title);
errors->dismiss_button().set_name ("ErrorLogCloseButton"); errors->dismiss_button().set_name ("ErrorLogCloseButton");
errors->signal_delete_event().connect (bind (ptr_fun (just_hide_it), (Gtk::Window *) errors)); errors->signal_delete_event().connect (bind (ptr_fun (just_hide_it), (Window *) errors));
register_thread (pthread_self(), X_("GUI")); register_thread (pthread_self(), X_("GUI"));
load_rcfile (rcfile);
_ok = true; _ok = true;
} }
@ -113,38 +115,76 @@ UI::load_rcfile (string path)
return -1; return -1;
} }
gtk_rc_parse (path.c_str()); RC rc (path.c_str());
Gtk::Label *a_widget1; /* have to pack widgets into a toplevel window so that styles will stick */
Gtk::Label *a_widget2;
Gtk::Label *a_widget3;
Gtk::Label *a_widget4;
a_widget1 = new Gtk::Label; Window temp_window (WINDOW_TOPLEVEL);
a_widget2 = new Gtk::Label; HBox box;
a_widget3 = new Gtk::Label; Label a_widget1;
a_widget4 = new Gtk::Label; Label a_widget2;
Label a_widget3;
Label a_widget4;
RefPtr<Gtk::Style> style;
RefPtr<TextBuffer> buffer (errors->text().get_buffer());
a_widget1->set_name ("FatalMessage"); box.pack_start (a_widget1);
a_widget1->ensure_style (); box.pack_start (a_widget2);
fatal_message_style = a_widget1->get_style(); box.pack_start (a_widget3);
box.pack_start (a_widget4);
a_widget2->set_name ("ErrorMessage"); error_ptag = buffer->create_tag();
a_widget2->ensure_style (); error_mtag = buffer->create_tag();
error_message_style = a_widget2->get_style(); fatal_ptag = buffer->create_tag();
fatal_mtag = buffer->create_tag();
warning_ptag = buffer->create_tag();
warning_mtag = buffer->create_tag();
info_ptag = buffer->create_tag();
info_mtag = buffer->create_tag();
a_widget3->set_name ("WarningMessage"); a_widget1.set_name ("FatalMessage");
a_widget3->ensure_style (); a_widget1.ensure_style ();
warning_message_style = a_widget3->get_style(); style = a_widget1.get_style();
a_widget4->set_name ("InfoMessage"); fatal_ptag->property_font_desc().set_value(style->get_font());
a_widget4->ensure_style (); fatal_ptag->property_foreground_gdk().set_value(style->get_fg(STATE_ACTIVE));
info_message_style = a_widget4->get_style(); fatal_ptag->property_background_gdk().set_value(style->get_bg(STATE_ACTIVE));
fatal_mtag->property_font_desc().set_value(style->get_font());
fatal_mtag->property_foreground_gdk().set_value(style->get_fg(STATE_NORMAL));
fatal_mtag->property_background_gdk().set_value(style->get_bg(STATE_NORMAL));
delete a_widget1; a_widget2.set_name ("ErrorMessage");
delete a_widget2; a_widget2.ensure_style ();
delete a_widget3; style = a_widget2.get_style();
delete a_widget4;
error_ptag->property_font_desc().set_value(style->get_font());
error_ptag->property_foreground_gdk().set_value(style->get_fg(STATE_ACTIVE));
error_ptag->property_background_gdk().set_value(style->get_bg(STATE_ACTIVE));
error_mtag->property_font_desc().set_value(style->get_font());
error_mtag->property_foreground_gdk().set_value(style->get_fg(STATE_NORMAL));
error_mtag->property_background_gdk().set_value(style->get_bg(STATE_NORMAL));
a_widget3.set_name ("WarningMessage");
a_widget3.ensure_style ();
style = a_widget3.get_style();
warning_ptag->property_font_desc().set_value(style->get_font());
warning_ptag->property_foreground_gdk().set_value(style->get_fg(STATE_ACTIVE));
warning_ptag->property_background_gdk().set_value(style->get_bg(STATE_ACTIVE));
warning_mtag->property_font_desc().set_value(style->get_font());
warning_mtag->property_foreground_gdk().set_value(style->get_fg(STATE_NORMAL));
warning_mtag->property_background_gdk().set_value(style->get_bg(STATE_NORMAL));
a_widget4.set_name ("InfoMessage");
a_widget4.ensure_style ();
style = a_widget4.get_style();
info_ptag->property_font_desc().set_value(style->get_font());
info_ptag->property_foreground_gdk().set_value(style->get_fg(STATE_ACTIVE));
info_ptag->property_background_gdk().set_value(style->get_bg(STATE_ACTIVE));
info_mtag->property_font_desc().set_value(style->get_font());
info_mtag->property_foreground_gdk().set_value(style->get_fg(STATE_NORMAL));
info_mtag->property_background_gdk().set_value(style->get_bg(STATE_NORMAL));
return 0; return 0;
} }
@ -190,7 +230,7 @@ UI::quit ()
void void
UI::do_quit () UI::do_quit ()
{ {
Gtk::Main::quit(); Main::quit();
} }
void void
@ -250,7 +290,7 @@ UI::call_slot_locked (sigc::slot<void> slot)
} }
void void
UI::set_tip (Gtk::Widget *w, const gchar *tip, const gchar *hlp) UI::set_tip (Widget *w, const gchar *tip, const gchar *hlp)
{ {
Request *req = get_request (SetTip); Request *req = get_request (SetTip);
@ -266,7 +306,7 @@ UI::set_tip (Gtk::Widget *w, const gchar *tip, const gchar *hlp)
} }
void void
UI::set_state (Gtk::Widget *w, Gtk::StateType state) UI::set_state (Widget *w, StateType state)
{ {
Request *req = get_request (StateChange); Request *req = get_request (StateChange);
@ -374,7 +414,7 @@ UI::setup_signal_pipe ()
if (pipe (signal_pipe)) { if (pipe (signal_pipe)) {
error << "UI: cannot create error signal pipe (" error << "UI: cannot create error signal pipe ("
<< strerror (errno) << ")" << std::strerror (errno) << ")"
<< endmsg; << endmsg;
return -1; return -1;
@ -383,7 +423,7 @@ UI::setup_signal_pipe ()
if (fcntl (signal_pipe[0], F_SETFL, O_NONBLOCK)) { if (fcntl (signal_pipe[0], F_SETFL, O_NONBLOCK)) {
error << "UI: cannot set O_NONBLOCK on " error << "UI: cannot set O_NONBLOCK on "
"signal read pipe (" "signal read pipe ("
<< strerror (errno) << ")" << std::strerror (errno) << ")"
<< endmsg; << endmsg;
return -1; return -1;
} }
@ -391,7 +431,7 @@ UI::setup_signal_pipe ()
if (fcntl (signal_pipe[1], F_SETFL, O_NONBLOCK)) { if (fcntl (signal_pipe[1], F_SETFL, O_NONBLOCK)) {
error << "UI: cannot set O_NONBLOCK on " error << "UI: cannot set O_NONBLOCK on "
"signal write pipe (" "signal write pipe ("
<< strerror (errno) << std::strerror (errno)
<< ")" << ")"
<< endmsg; << endmsg;
return -1; return -1;
@ -588,25 +628,29 @@ UI::receive (Transmitter::Channel chn, const char *str)
void void
UI::process_error_message (Transmitter::Channel chn, const char *str) UI::process_error_message (Transmitter::Channel chn, const char *str)
{ {
Glib::RefPtr<Gtk::Style> style; RefPtr<Style> style;
RefPtr<TextBuffer::Tag> ptag;
RefPtr<TextBuffer::Tag> mtag;
char *prefix; char *prefix;
size_t prefix_len; size_t prefix_len;
bool fatal_received = false; bool fatal_received = false;
#ifndef OLD_STYLE_ERRORS #ifndef OLD_STYLE_ERRORS
PopUp* popup = new PopUp (Gtk::WIN_POS_CENTER, 0, true); PopUp* popup = new PopUp (WIN_POS_CENTER, 0, true);
#endif #endif
switch (chn) { switch (chn) {
case Transmitter::Fatal: case Transmitter::Fatal:
prefix = "[FATAL]: "; prefix = "[FATAL]: ";
style = fatal_message_style; ptag = fatal_ptag;
mtag = fatal_mtag;
prefix_len = 9; prefix_len = 9;
fatal_received = true; fatal_received = true;
break; break;
case Transmitter::Error: case Transmitter::Error:
#if OLD_STYLE_ERRORS #if OLD_STYLE_ERRORS
prefix = "[ERROR]: "; prefix = "[ERROR]: ";
style = error_message_style; ptag = error_ptag;
mtag = error_mtag;
prefix_len = 9; prefix_len = 9;
#else #else
popup->set_name ("ErrorMessage"); popup->set_name ("ErrorMessage");
@ -618,7 +662,8 @@ UI::process_error_message (Transmitter::Channel chn, const char *str)
case Transmitter::Info: case Transmitter::Info:
#if OLD_STYLE_ERRORS #if OLD_STYLE_ERRORS
prefix = "[INFO]: "; prefix = "[INFO]: ";
style = info_message_style; ptag = info_ptag;
mtag = info_mtag;
prefix_len = 8; prefix_len = 8;
#else #else
popup->set_name ("InfoMessage"); popup->set_name ("InfoMessage");
@ -631,7 +676,8 @@ UI::process_error_message (Transmitter::Channel chn, const char *str)
case Transmitter::Warning: case Transmitter::Warning:
#if OLD_STYLE_ERRORS #if OLD_STYLE_ERRORS
prefix = "[WARNING]: "; prefix = "[WARNING]: ";
style = warning_message_style; ptag = warning_ptag;
mtag = warning_mtag;
prefix_len = 11; prefix_len = 11;
#else #else
popup->set_name ("WarningMessage"); popup->set_name ("WarningMessage");
@ -652,7 +698,7 @@ UI::process_error_message (Transmitter::Channel chn, const char *str)
handle_fatal (str); handle_fatal (str);
} else { } else {
display_message (prefix, prefix_len, style, str); display_message (prefix, prefix_len, ptag, mtag, str);
if (!errors->is_visible()) { if (!errors->is_visible()) {
toggle_errors(); toggle_errors();
@ -666,7 +712,7 @@ void
UI::toggle_errors () UI::toggle_errors ()
{ {
if (!errors->is_visible()) { if (!errors->is_visible()) {
errors->set_position (Gtk::WIN_POS_MOUSE); errors->set_position (WIN_POS_MOUSE);
errors->show (); errors->show ();
} else { } else {
errors->hide (); errors->hide ();
@ -674,22 +720,10 @@ UI::toggle_errors ()
} }
void void
UI::display_message (const char *prefix, gint prefix_len, Glib::RefPtr<Gtk::Style> style, const char *msg) UI::display_message (const char *prefix, gint prefix_len, RefPtr<TextBuffer::Tag> ptag, RefPtr<TextBuffer::Tag> mtag, const char *msg)
{ {
Glib::RefPtr<Gtk::TextBuffer::Tag> ptag = Gtk::TextBuffer::Tag::create(); RefPtr<TextBuffer> buffer (errors->text().get_buffer());
Glib::RefPtr<Gtk::TextBuffer::Tag> mtag = Gtk::TextBuffer::Tag::create();
Pango::FontDescription pfont(style->get_font());
ptag->property_font_desc().set_value(pfont);
ptag->property_foreground_gdk().set_value(style->get_fg(Gtk::STATE_ACTIVE));
ptag->property_background_gdk().set_value(style->get_bg(Gtk::STATE_ACTIVE));
Pango::FontDescription mfont (style->get_font());
mtag->property_font_desc().set_value(mfont);
mtag->property_foreground_gdk().set_value(style->get_fg(Gtk::STATE_NORMAL));
mtag->property_background_gdk().set_value(style->get_bg(Gtk::STATE_NORMAL));
Glib::RefPtr<Gtk::TextBuffer> buffer (errors->text().get_buffer());
buffer->insert_with_tag(buffer->end(), prefix, ptag); buffer->insert_with_tag(buffer->end(), prefix, ptag);
buffer->insert_with_tag(buffer->end(), msg, mtag); buffer->insert_with_tag(buffer->end(), msg, mtag);
buffer->insert_with_tag(buffer->end(), "\n", mtag); buffer->insert_with_tag(buffer->end(), "\n", mtag);
@ -700,10 +734,10 @@ UI::display_message (const char *prefix, gint prefix_len, Glib::RefPtr<Gtk::Styl
void void
UI::handle_fatal (const char *message) UI::handle_fatal (const char *message)
{ {
Gtk::Window win (Gtk::WINDOW_POPUP); Window win (WINDOW_POPUP);
Gtk::VBox packer; VBox packer;
Gtk::Label label (message); Label label (message);
Gtk::Button quit (_("Press To Exit")); Button quit (_("Press To Exit"));
win.set_default_size (400, 100); win.set_default_size (400, 100);
@ -712,7 +746,7 @@ UI::handle_fatal (const char *message)
title += ": Fatal Error"; title += ": Fatal Error";
win.set_title (title); win.set_title (title);
win.set_position (Gtk::WIN_POS_MOUSE); win.set_position (WIN_POS_MOUSE);
win.add (packer); win.add (packer);
packer.pack_start (label, true, true); packer.pack_start (label, true, true);
@ -738,7 +772,7 @@ UI::popup_error (const char *text)
return; return;
} }
pup = new PopUp (Gtk::WIN_POS_MOUSE, 0, true); pup = new PopUp (WIN_POS_MOUSE, 0, true);
pup->set_text (text); pup->set_text (text);
pup->touch (); pup->touch ();
} }
@ -761,7 +795,7 @@ UI::flush_pending ()
} }
bool bool
UI::just_hide_it (GdkEventAny *ev, Gtk::Window *win) UI::just_hide_it (GdkEventAny *ev, Window *win)
{ {
win->hide_all (); win->hide_all ();
return true; return true;
@ -772,7 +806,7 @@ UI::get_color (const string& prompt, bool& picked, Gdk::Color* initial)
{ {
Gdk::Color color; Gdk::Color color;
Gtk::ColorSelectionDialog color_dialog (prompt); ColorSelectionDialog color_dialog (prompt);
color_dialog.set_modal (true); color_dialog.set_modal (true);
color_dialog.get_cancel_button()->signal_clicked().connect (bind (mem_fun (*this, &UI::color_selection_done), false)); color_dialog.get_cancel_button()->signal_clicked().connect (bind (mem_fun (*this, &UI::color_selection_done), false));
@ -787,7 +821,7 @@ UI::get_color (const string& prompt, bool& picked, Gdk::Color* initial)
color_picked = false; color_picked = false;
picked = false; picked = false;
Gtk::Main::run(); Main::run();
color_dialog.hide_all (); color_dialog.hide_all ();
@ -807,12 +841,12 @@ void
UI::color_selection_done (bool status) UI::color_selection_done (bool status)
{ {
color_picked = status; color_picked = status;
Gtk::Main::quit (); Main::quit ();
} }
bool bool
UI::color_selection_deleted (GdkEventAny *ev) UI::color_selection_deleted (GdkEventAny *ev)
{ {
Gtk::Main::quit (); Main::quit ();
return true; return true;
} }

View file

@ -28,6 +28,7 @@
#include <pthread.h> #include <pthread.h>
#include <gtkmm/widget.h> #include <gtkmm/widget.h>
#include <gtkmm/style.h> #include <gtkmm/style.h>
#include <gtkmm/textbuffer.h>
#include <gtkmm/main.h> #include <gtkmm/main.h>
#include <gtkmm/tooltips.h> #include <gtkmm/tooltips.h>
#include <gdkmm/color.h> #include <gdkmm/color.h>
@ -103,7 +104,9 @@ class UI : public AbstractUI
protected: protected:
virtual void handle_fatal (const char *); virtual void handle_fatal (const char *);
virtual void display_message (const char *prefix, gint prefix_len, virtual void display_message (const char *prefix, gint prefix_len,
Glib::RefPtr<Gtk::Style> style, const char *msg); Glib::RefPtr<Gtk::TextBuffer::Tag> ptag,
Glib::RefPtr<Gtk::TextBuffer::Tag> mtag,
const char *msg);
/* stuff to invoke member functions in another /* stuff to invoke member functions in another
thread so that we can keep the GUI running. thread so that we can keep the GUI running.
@ -160,13 +163,14 @@ class UI : public AbstractUI
Gtk::Main *theMain; Gtk::Main *theMain;
Gtk::Tooltips *tips; Gtk::Tooltips *tips;
TextViewer *errors; TextViewer *errors;
Glib::RefPtr<Gtk::TextBuffer::Tag> error_ptag;
/* error message display styles */ Glib::RefPtr<Gtk::TextBuffer::Tag> error_mtag;
Glib::RefPtr<Gtk::TextBuffer::Tag> fatal_ptag;
Glib::RefPtr<Gtk::Style> error_message_style; Glib::RefPtr<Gtk::TextBuffer::Tag> fatal_mtag;
Glib::RefPtr<Gtk::Style> info_message_style; Glib::RefPtr<Gtk::TextBuffer::Tag> info_ptag;
Glib::RefPtr<Gtk::Style> warning_message_style; Glib::RefPtr<Gtk::TextBuffer::Tag> info_mtag;
Glib::RefPtr<Gtk::Style> fatal_message_style; Glib::RefPtr<Gtk::TextBuffer::Tag> warning_ptag;
Glib::RefPtr<Gtk::TextBuffer::Tag> warning_mtag;
int signal_pipe[2]; int signal_pipe[2];
PBD::Lock request_buffer_map_lock; PBD::Lock request_buffer_map_lock;