Ticket #2684: mc-4.8.1-cursor-after-resize-try3.patch

src/filemanager/panel.c

@@ -1139 +1139 @@
 }
 
 /* --------------------------------------------------------------------------------------------- */
-/** To be used only by long_frame and full_frame to adjust top_file */
+
+/* Returns the number of items in the given panel */
+static int
+ITEMS (WPanel * p)
+{
+    if (p->split)
+        return llines (p) * 2;
+    else
+        return llines (p);
+}
+
+/* --------------------------------------------------------------------------------------------- */
 
 static void
 adjust_top_file (WPanel * panel)
 {
-    int old_top = panel->top_file;
+    int items = ITEMS (panel);
 
-    if (panel->selected - old_top > llines (panel))
+    /* If there's not enough room to display all files (count > items) then:
+       - top_file has to be in the range [selected-items+1, selected] so that
+         the selected file is visible,
+       - top_file should be in the range [0, count-items] so that there's
+         no empty space wasted,
+       - within these ranges, adjust it by as little as possible.
+       If all files fit (count <= items) then top_file should be set to 0
+       to show them all, this is done by the last two checks. */
+    if (panel->top_file > panel->selected)
         panel->top_file = panel->selected;
-    if (old_top - panel->count > llines (panel))
-        panel->top_file = panel->count - llines (panel);
+
+    if (panel->top_file < panel->selected - items + 1)
+        panel->top_file = panel->selected - items + 1;
+
+    if (panel->top_file > panel->count - items)
+        panel->top_file = panel->count - items;
+
+    if (panel->top_file < 0)
+        panel->top_file = 0;
 }
 
 /* --------------------------------------------------------------------------------------------- */
@@ -1311 +1337 @@
     int side, width;
     GString *format_txt;
 
-    if (!panel->split)
-        adjust_top_file (panel);
+    adjust_top_file (panel);
 
     widget_erase (&panel->widget);
     show_dir (panel);
@@ -1743 +1768 @@
 
 /* --------------------------------------------------------------------------------------------- */
 
-/* Returns the number of items in the given panel */
-static int
-ITEMS (WPanel * p)
-{
-    if (p->split)
-        return llines (p) * 2;
-    else
-        return llines (p);
-}
-
-/* --------------------------------------------------------------------------------------------- */
-
 static void
 unselect_item (WPanel * panel)
 {
@@ -3914 +3927 @@
 void
 select_item (WPanel * panel)
 {
-    int items = ITEMS (panel);
-
     /* Although currently all over the code we set the selection and
        top file to decent values before calling select_item, I could
        forget it someday, so it's better to do the actual fitting here */
 
-    if (panel->top_file < 0)
-        panel->top_file = 0;
-
     if (panel->selected < 0)
         panel->selected = 0;
 
     if (panel->selected > panel->count - 1)
         panel->selected = panel->count - 1;
 
-    if (panel->top_file > panel->count - 1)
-        panel->top_file = panel->count - 1;
-
-    if ((panel->count - panel->top_file) < items)
-    {
-        panel->top_file = panel->count - items;
-        if (panel->top_file < 0)
-            panel->top_file = 0;
-    }
-
-    if (panel->selected < panel->top_file)
-        panel->top_file = panel->selected;
-
-    if ((panel->selected - panel->top_file) >= items)
-        panel->top_file = panel->selected - items + 1;
+    adjust_top_file (panel);
 
     panel->dirty = 1;