File Browser: Scroll view on normal selection too
authorJulian Eisel <eiseljulian@gmail.com>
Sat, 19 Sep 2015 02:24:48 +0000 (04:24 +0200)
committerJulian Eisel <eiseljulian@gmail.com>
Sat, 19 Sep 2015 02:24:48 +0000 (04:24 +0200)
Adjusts view after mouse/border selection if some selected items are out of view bounds.
To get as much of the selection into view as possible, this adjusts view first for the last, then for the first element in the selection.
Also, if region is pretty small, view adjustment is skipped, as otherwise the view is focused on the first element only, which isn't really useful IMHO.

Maybe not so nice: Since we do two view alignment iterations, UI_view2d_curRect_validate, which is a rather big function *might* be called twice under certain circumstances (border select & total size of selected elements is exceeds view bounds). I think that's totally acceptable though.

source/blender/editors/space_file/file_ops.c

index eb0ad4bbff977d2ab335d617311a2a34f0e6d4df..8347ad872589ad137912102253aaf72a39b02a44 100644 (file)
@@ -305,6 +305,20 @@ static FileSelect file_select(bContext *C, const rcti *rect, FileSelType select,
        if (select != FILE_SEL_ADD && !file_is_any_selected(sfile->files)) {
                sfile->params->active_file = -1;
        }
+       else {
+               ARegion *ar = CTX_wm_region(C);
+               const FileLayout *layout = ED_fileselect_get_layout(sfile, ar);
+
+               /* Adjust view to display selection. Doing iterations for first and last
+                * selected item makes view showing as much of the selection possible.
+                * Not really useful if tiles are (almost) bigger than viewbounds though. */
+               if (((layout->flag & FILE_LAYOUT_HOR) && ar->winx > (1.2f * layout->tile_w)) ||
+                   ((layout->flag & FILE_LAYOUT_VER) && ar->winy > (2.0f * layout->tile_h)))
+               {
+                       file_ensure_inside_viewbounds(ar, sfile, sel.last);
+                       file_ensure_inside_viewbounds(ar, sfile, sel.first);
+               }
+       }
 
        /* update operator for name change event */
        file_draw_check(C);