AccordionAll 2 scrolls the screen right unnecessarily sometimes

[svermeulen]

The biggest source of frustration for me with this plugin is that often times I'll open a new vertical split only to find that the previous split I was in has been scrolled to the point where I can't really see the text I was working with, forcing me to jump back to the previous split and adjust the horizontal scroll.

Repro steps:

• Open vanilla vim + vim-accordion
• Execute set nowrap
• Execute AccordionAll 2
• Add some text to a new file such that your cursor is in the horizontal middle
• Create a new split with an existing text file by executing :vsp [path to test file]
• Notice that you can no longer see the beginning of the line you were at in the previous split
• Execute AccordionStop
• Repeat by closing the new split, going back to the previous split, fixing the scroll, and then executing :vsp [path to test file]
• Notice that you can still see beginning of the text in the previous split unlike with AccordionAll on

[mattboehm]

Thanks for reporting, I'll look into this. When windows get squished, their scroll changes so I try to use winsaveview() and winrestview() to work around this issue, but there must be a bug with it.

[svermeulen]

I also noticed that this issue occurs when creating a new tab. The following fixes that:

    autocmd WinEnter * normal! 80zh

But doesn't address the problem outlined above when creating new splits (since it's the previous window with the bad scroll not the new one)

[svermeulen]

Found a better workaround in the function accordion#Accordion

"jump to prevwin and back so that window history is preserved
execute prevwin "wincmd w"
normal! 100zh
execute curwin "wincmd w"
normal! 100zh

Obviously a bit hacky though.

[mattboehm]

Thanks, that helps. My main things to consider are if this alienates people that had the view intentionally scrolled to the right. I'd also need to ensure that the solution works with any number of windows and accordion size. If the best end solution still feels a bit hacky though, worst case, I make it an optional setting.