Movie list view: remove unneeded sort and filter wrappers for the list store

[cinaest] / src / movie-list-view.vala

diff --git a/src/movie-list-view.vala b/src/movie-list-view.vala
index 205f5ce..87e67c1 100644 (file)
--- a/src/movie-list-view.vala
+++ b/src/movie-list-view.vala
@@ -22,7 +22,6 @@ using Hildon;
 public class MovieListView : PannableArea {
        public MovieListStore store;
        TreeView tree;
-       public TreeSortable sorted_store;
 
        private bool more_movies_available;
        private CellRendererText title_renderer;
@@ -35,14 +34,14 @@ public class MovieListView : PannableArea {
        public MovieListView (Gtk.Window window, bool show_date = false) {
                store = new MovieListStore ();
 
-               // Add filter wrapper
-               var filtered_store = new TreeModelFilter (store, null);
-
-               // Add sort wrapper
-               sorted_store = new TreeModelSort.with_model (filtered_store);
+               Gdk.Color color;
+               window.ensure_style ();
+               if (window.style.lookup_color ("SecondaryTextColor", out color)) {
+                       store.year_markup = "<span size=\"small\" fgcolor=\"%s\">(%%d)</span>".printf (color.to_string ());
+               }
 
                // Tree View
-               tree = (TreeView) Hildon.gtk_tree_view_new_with_model (UIMode.NORMAL, sorted_store);
+               tree = (TreeView) Hildon.gtk_tree_view_new_with_model (UIMode.NORMAL, store);
                tree.set_headers_visible (false);
 
                add (tree);
@@ -91,17 +90,14 @@ public class MovieListView : PannableArea {
                tree.append_column (title_column);
 
                // Sort by title
-               sorted_store.set_sort_column_id (MovieListStore.Columns.TITLE, SortType.ASCENDING);
+               store.set_sort_column_id (MovieListStore.Columns.TITLE, SortType.ASCENDING);
 
                // Year column
-               var renderer = new CellRendererText ();
                var year_column = new TreeViewColumn ();
-               year_column.set_title (_("Rating"));
+               year_column.set_title (_("Year"));
                year_column.set_sort_column_id (MovieListStore.Columns.YEAR);
                year_column.set_reorderable (false);
                year_column.set_sort_order (SortType.DESCENDING);
-               year_column.pack_start (renderer, true);
-               year_column.set_cell_data_func (renderer, year_data_func);
                tree.append_column (year_column);
 
                // Rating column
@@ -200,26 +196,21 @@ public class MovieListView : PannableArea {
 
        private void title_data_func (CellLayout cell_layout, CellRenderer cell, TreeModel model, TreeIter iter) {
                Movie movie;
+               string markup;
 
-               model.get (iter, MovieListStore.Columns.MOVIE, out movie);
-               title_renderer.text = movie.title;
+               model.get (iter, MovieListStore.Columns.MOVIE, out movie,
+                                MovieListStore.Columns.MARKUP, out markup);
+               title_renderer.markup = markup;
                secondary_renderer.text = movie.secondary;
        }
 
-       private void year_data_func (CellLayout cell_layout, CellRenderer cell, TreeModel model, TreeIter iter) {
-               int year;
-
-               model.get (iter, MovieListStore.Columns.YEAR, out year);
-               ((CellRendererText) cell).text = (year > 0) ? year.to_string () : "";
-       }
-
        private void rating_data_func (CellLayout cell_layout, CellRenderer cell, TreeModel model, TreeIter iter) {
-               int rating;
+               string rating;
                Movie movie;
 
                model.get (iter, MovieListStore.Columns.RATING, out rating,
                                 MovieListStore.Columns.MOVIE, out movie);
-               rating_renderer.text = (rating > 0) ? "%d.%d".printf (rating / 10, rating % 10) : "";
+               rating_renderer.text = rating;
                if (movie.julian_date != 0) {
                        var date = Date ();
                        date.set_julian (movie.julian_date);