Hello.

As mentioned in message #10, the problem arises when wrapping lines and
pager_index_lines>0.

The reason is there's no sidebar divider refresh when redraw does not 
have the index redraw bit set and there's a mini-index.

These sidebar patch-sets would benefit from more comprehensive TLC but
for the time being, the attached patches-on-patches against:

a) patches/mutt-patched/sidebar (in 1.5.21-6.2+deb7u2)
b) patches/mutt-patched/sidebar.patch (in 1.5.23-1)

ensure a redraw occurs with or without a mini-index.

Feel free to adopt them.

--mancha
--- a/patches/mutt-patched/sidebar
+++ b/patches/mutt-patched/sidebar
@@ -923,11 +923,10 @@ in the "mutt-patched" package.
        }
        BKGDSET (MT_COLOR_NORMAL);
        SETCOLOR (MT_COLOR_NORMAL);
-@@ -1826,18 +1830,23 @@
+@@ -1826,18 +1830,21 @@
        /* redraw the pager_index indicator, because the
         * flags for this message might have changed. */
        menu_redraw_current (index);
-+      draw_sidebar(MENU_PAGER);
  
        /* print out the index status bar */
        menu_status_line (buffer, sizeof (buffer), index, NONULL(Status));
@@ -942,14 +941,13 @@ in the "mutt-patched" package.
        BKGDSET (MT_COLOR_NORMAL);
      }
  
-+    /* if we're not using the index, update every time */
-+    if ( index == 0 )
-+      draw_sidebar(MENU_PAGER);
++    /* Update every time */
++    draw_sidebar(0);
 +
      redraw = 0;
  
      if (option(OPTBRAILLEFRIENDLY)) {
-@@ -2769,6 +2778,13 @@
+@@ -2769,6 +2776,13 @@
        mutt_what_key ();
        break;
  
--- a/patches/mutt-patched/sidebar.patch
+++ b/patches/mutt-patched/sidebar.patch
@@ -1055,11 +1033,10 @@ index 7b61266..469efe4 100644
        }
        NORMAL_COLOR;
        if (option(OPTXTERMSETTITLES))
-@@ -1826,16 +1830,21 @@ mutt_pager (const char *banner, const char *fname, int 
flags, pager_t *extra)
+@@ -1826,16 +1830,19 @@ mutt_pager (const char *banner, const char *fname, int 
flags, pager_t *extra)
        /* redraw the pager_index indicator, because the
         * flags for this message might have changed. */
        menu_redraw_current (index);
-+      draw_sidebar(MENU_PAGER);
  
        /* print out the index status bar */
        menu_status_line (buffer, sizeof (buffer), index, NONULL(Status));
@@ -1072,14 +1049,13 @@ index 7b61266..469efe4 100644
        NORMAL_COLOR;
      }
  
-+    /* if we're not using the index, update every time */
-+    if ( index == 0 )
-+      draw_sidebar(MENU_PAGER);
++    /* Update every time */
++    draw_sidebar(0);
 +
      redraw = 0;
  
      if (option(OPTBRAILLEFRIENDLY)) {
-@@ -2776,6 +2785,13 @@ search_next:
+@@ -2776,6 +2783,13 @@ search_next:
        mutt_what_key ();
        break;
  

Reply via email to