Changeset 66412030f4aeae129a0a9dd502638f834481454d

Timestamp:

04/07/07 23:30:54 (6 years ago)

Author:

Philipp Kern <phil@…>

Parents:

852b5aeb311ca8a9d90b790d791aca0160560178

Children:

24a527a2ffba5e33a2fe7375535d69f3357e4178

git-committer:

Philipp Kern <phil@0x539.de> / 2007-04-07T21:30:54Z+0000

Message:

2007-04-07 Philipp Kern <phil@…>

• inc/userlist.hpp:
• src/userlist.hpp: allow doubleclicks on documents in the userlist to subscribe to them [fixes #225]

Files:

• ChangeLog (modified) (1 diff)
• inc/userlist.hpp (modified) (2 diffs)
• src/userlist.cpp (modified) (6 diffs)

ChangeLog

@@ +1 @@
+2007-04-07  Philipp Kern <phil@0x539.de>
+
+        * inc/userlist.hpp:
+        * src/userlist.hpp: allow doubleclicks on documents in the userlist
+        to subscribe to them [fixes #225]
+
 2007-04-07  Philipp Kern <phil@0x539.de>
 

inc/userlist.hpp

@@ -43 +43 @@
                 Gtk::TreeModelColumn<Glib::RefPtr<Gdk::Pixbuf> > icon;
                 Gtk::TreeModelColumn<Glib::ustring> text;
+                Gtk::TreeModelColumn<LocalDocumentInfo*> info;
                 // TODO: Column with pointer that holds reference to obby::user?
         };
@@ -66 +67 @@
 
         void on_user_subscribe(const obby::user& user,
-                               const LocalDocumentInfo& info);
+                               LocalDocumentInfo& info);
         void on_user_unsubscribe(const obby::user& user,
                                  const LocalDocumentInfo& info);
+
+        void on_row_activated(const Gtk::TreePath& path,
+                              Gtk::TreeViewColumn* column);
 
         Header& m_header;

src/userlist.cpp

@@ -63 +63 @@
         add(icon);
         add(text);
+        add(info);
 }
 
@@ -106 +107 @@
         m_tree_view.get_selection()->set_mode(Gtk::SELECTION_NONE);
         m_tree_view.set_headers_visible(false);
+        m_tree_view.signal_row_activated().connect(
+                sigc::mem_fun(*this, &UserList::on_row_activated) );
 
         m_scrolled_wnd.set_shadow_type(Gtk::SHADOW_IN);
@@ -174 +177 @@
                         on_user_subscribe(
                                 user,
-                                dynamic_cast<const LocalDocumentInfo&>(
+                                dynamic_cast<LocalDocumentInfo&>(
                                         *iter
                                 )
@@ -240 +243 @@
 
 void Gobby::UserList::on_user_subscribe(const obby::user& user,
-                                        const LocalDocumentInfo& info)
+                                        LocalDocumentInfo& info)
 {
         Gtk::TreeIter iter = find_iter(m_iter_online, user.get_name() );
@@ -253 +256 @@
 
         (*doc)[m_tree_cols.text] = info.get_title();
+
+        (*doc)[m_tree_cols.info] = &info;
 }
 
@@ -288 +293 @@
                 iter = m_tree_data->erase(iter);
 }
+
+void Gobby::UserList::on_row_activated(const Gtk::TreePath& path,
+                                       Gtk::TreeViewColumn* columns)
+{
+        Gtk::TreeIter tree_iter = m_tree_data->get_iter(path);
+        LocalDocumentInfo* info = (*tree_iter)[m_tree_cols.info];
+
+        if(info == NULL)
+                // The selected row is a user, not a document.
+                return;
+
+        if(Gobby::is_subscribable(*info) &&
+           info->get_subscription_state() == Gobby::LocalDocumentInfo::UNSUBSCRIBED)
+        {
+                info->subscribe();
+        }
+}
+