Editor tab ‘blends’ into minimap if it is on left and multiple panes are open
Closed this issue · 11 comments
I prefer my minimap on the left but this is what happens to it:
It doesn't always happen, it just happens sometimes, specially when having multiple panes open.
When I move the minimap to the right side of the code editor window (which I don't like), the problem goes away.
1.54.0
Operating System: Manjaro Linux
KDE Plasma Version: 5.20.5
KDE Frameworks Version: 5.78.0
Qt Version: 5.15.2
Kernel Version: 5.10.7-3-MANJARO
OS Type: 64-bit
Processors: 16 × AMD Ryzen 7 2700X Eight-Core Processor
Memory: 31.4 GiB of RAM
Graphics Processor: Radeon RX 580 Series
I am unable to reproduce this. Is there any way you can get more specific about when it happens?
I wish I could tell you. I cannot reproduce this problem myself but it happens all the time.
To escape this issue, I had switched the minimap to the right side of the editor. I've switched it back to the left (the setting under which this problem occurs) and I'm going to wait for it to happen then I'll report back here with more details.
At this point, as far as I can tell, the necessary conditions are:
-minimap on the left.
-2 editor panes at least.
-resizing the panes. I use a package called hey-pane that enlarges the current pane for you, just the way you would manually. hey-pane itself is not a necessary condition because the problem occurs without the use of hey-pane as well.
I think we should issue a notification about the issue when multiple panes are open and minimap is on left, and suggest moving the Minimap to the right if they want to avoid this issue.
I used a crude hack for moving the scrollbar to the right when Minimap becomes hidden by the toggle command. This enforces the placement of the borders. However, this is a very bad solution for the regular situation. I prefer letting the user know about this limitation instead of trying to hack it in a crude way.
Line 171 in 64ee4b0
Ah, so I'm stuck with minimap on the right :/
In any case, I can't live without this package. Keep up the good work guys! 👍
I'll take a look at this tonight and see if there is an easy fix.
I see the problem is that minimap calculates the width and tries to move the editor to the right by that width. The problem is when there are multiple minimaps they might each have different widths but it has to choose one.
This seems like it can be solved by calculating each visible minimap on resize and compensating editors differently. I'll work on this in the next few days.
Actually that was easier than I thought. #783 should fix this. I just need to test it a little more to make sure it doesn't break anything else.
I'd love to help in testing if you need.
Sure, You can install the branch https://github.com/UziTech/minimap/tree/left-width to test it out
- uninstall minimap
- clone https://github.com/UziTech/minimap/tree/left-width
- run
npm i
in the cloned folder - run
apm link
🎉 This issue has been resolved in version 4.39.8 🎉
The release is available on:
Your semantic-release bot 📦🚀