FListView now has a sort indicator (▼/▲) to display the sort order

.codedocs

@@ -1,4 +1,4 @@
 PROJECT_NAME = "The Final Cut"
-EXCLUDE = debian, icon, logo, m4, scripts, examples
+EXCLUDE = debian, doc, icon, logo, m4, scripts, examples
 EXCLUDE_PATTERNS = */test/*
 

ChangeLog

@@ -1,3 +1,6 @@
+2018-11-10  Markus Gans  <guru.mail@muenster.de>
+	* FListView now has a sort indicator to display the sort order
+
 2018-11-07  Markus Gans  <guru.mail@muenster.de>
 	* Use new type FColor for color values
 

examples/listview.cpp

@@ -94,9 +94,12 @@ Listview::Listview (finalcut::FWidget* parent)
 
   // Sort in ascending order by the 1st column
   listView.setColumnSort (1, finalcut::fc::ascending);
-  // The sorting occurs later automatically at insert().
+  // Sorting follows later automatically on insert().
   // Otherwise you could start the sorting directly with sort()
 
+  // Allways show the sort indicator (▼/▲)
+  listView.hideSortIndicator(false);
+
   // Populate FListView with a list of items
   populate();
 

src/fdialog.cpp

@@ -1615,7 +1615,8 @@ void FDialog::resizeMouseUpMove (mouseStates& ms, bool mouse_up)
       else
         h = int(getMaxHeight()) - getTermY() + y2_offset + 1;
 
-      setSize (std::size_t(w), std::size_t(h));
+      setSize ( ( w >= 0) ? std::size_t(w) : 0
+              , ( h >= 0) ? std::size_t(h) : 0 );
     }
 
     if ( mouse_up )

src/flistview.cpp

@@ -610,6 +610,7 @@ FListView::FListView (FWidget* parent)
   , scroll_distance(1)
   , scroll_timer(false)
   , tree_view(false)
+  , hide_sort_indicator(false)
   , clicked_expander_pos(-1, -1)
   , xoffset(0)
   , nf_offset(0)
@@ -1766,37 +1767,79 @@ inline FString FListView::getLinePrefix ( const FListViewItem* item
   return line;
 }
 
+//----------------------------------------------------------------------
+inline void FListView::drawSortIndicator ( std::size_t& length
+                                         , std::size_t  column_width )
+{
+  if ( length >= column_width )
+    return;
+
+  setColor();
+  length++;
+
+  if ( sort_order == fc::ascending )
+    headerline << wchar_t(fc::BlackDownPointingTriangle);  // ▼
+  else if ( sort_order == fc::descending )
+    headerline << wchar_t(fc::BlackUpPointingTriangle);    // ▲
+
+  if ( length < column_width  )
+  {
+    length++;
+    headerline << ' ';
+  }
+}
+
+//----------------------------------------------------------------------
+inline void FListView::drawHeaderLine (std::size_t length)
+{
+  setColor();
+  const FString line (length, wchar_t(fc::BoxDrawingsHorizontal));
+  headerline << line;  // horizontal line
+}
+
 //----------------------------------------------------------------------
 void FListView::drawColumnText (headerItems::const_iterator& iter)
 {
   // Print lable text
   static const std::size_t leading_space = 1;
-  static const std::size_t trailing_space = 1;
   const FString& text = iter->name;
-  std::size_t width = std::size_t(iter->width);
   FString txt = " " + text;
+  std::size_t width = std::size_t(iter->width);
   std::size_t txt_length = txt.getLength();
+  std::size_t length = 0;
   std::size_t column_width = leading_space + width;
+  headerItems::const_iterator first = header.begin();
+  int column = std::distance(first, iter) + 1;
+  bool isSortIndicator = bool ( sort_column == column
+                             && ! hide_sort_indicator );
 
   if ( isEnabled() )
     setColor (wc.label_emphasis_fg, wc.label_bg);
   else
     setColor (wc.label_inactive_fg, wc.label_inactive_bg);
 
+  if ( isSortIndicator && txt_length >= column_width - 1 && txt_length > 1 )
+  {
+    txt_length = column_width - 2;
+    txt = txt.left(txt_length);
+  }
+
   if ( txt_length <= column_width )
   {
+    length = txt_length;
     headerline << txt;
 
-    if ( txt_length < column_width )
+    if ( length < column_width )
-      headerline << ' ';  // trailing space
-
-    if ( txt_length + trailing_space < column_width )
     {
-      setColor();
+      length++;
-      const FString line ( column_width - trailing_space - txt_length
+      headerline << ' ';  // trailing space
-                         , wchar_t(fc::BoxDrawingsHorizontal) );
-      headerline << line;  // horizontal line
     }
+
+    if ( isSortIndicator )
+      drawSortIndicator (length, column_width );
+
+    if ( length < column_width )
+      drawHeaderLine (column_width - length);
   }
   else
     drawColumnEllipsis (iter, text);  // Print ellipsis

src/include/final/flistview.h

@@ -283,6 +283,7 @@ class FListView : public FWidget
     void                 setUserAscendingCompare (Compare);
     template <typename Compare>
     void                 setUserDescendingCompare (Compare);
+    void                 hideSortIndicator (bool);
     bool                 setTreeView (bool);
     bool                 setTreeView();
     bool                 unsetTreeView();
@@ -356,7 +357,9 @@ class FListView : public FWidget
     void                 drawListLine (const FListViewItem*, bool, bool);
     void                 setLineAttributes (bool, bool);
     FString              getLinePrefix (const FListViewItem*, std::size_t);
+    void                 drawSortIndicator (std::size_t&, std::size_t);
     void                 drawColumnText (headerItems::const_iterator&);
+    void                 drawHeaderLine (std::size_t);
     void                 drawColumnEllipsis ( headerItems::const_iterator&
                                             , const FString& );
     void                 updateDrawing (bool, bool);
@@ -410,6 +413,7 @@ class FListView : public FWidget
     int                  scroll_distance;
     bool                 scroll_timer;
     bool                 tree_view;
+    bool                 hide_sort_indicator;
     FPoint               clicked_expander_pos;
     int                  xoffset;
     int                  nf_offset;
@@ -477,6 +481,10 @@ template <typename Compare>
 inline void FListView::setUserDescendingCompare (Compare cmp)
 { user_defined_descending = cmp; }
 
+//----------------------------------------------------------------------
+inline void FListView::hideSortIndicator (bool hide)
+{ hide_sort_indicator = hide; }
+
 //----------------------------------------------------------------------
 inline bool FListView::setTreeView (bool on)
 { return (tree_view = on); }

src/include/final/fobject.h

@@ -35,6 +35,10 @@
   #error "Only <final/final.h> can be included directly."
 #endif
 
+#if __cplusplus < 199711L
+  #error "Your C++ compiler does not support the C++98 standard!"
+#endif
+
 #include <sys/time.h>  // need for gettimeofday
 #include <cstdlib>
 #include <cstring>