Stopped working in 0.177.0

[thislooksfun]

After updating to Atom 0.177.0 and Minimap 4.1.0, the minimap stopped working.
Settings:

Screenshot:

It's probably something I have done, but I see no reason why it wouldn't be working.

[fundon]

It's working on my machine.

Disable other plugins and try again.

[abe33]

FWIW have you tried disabling the hardware acceleration?

[thislooksfun]

I tried both disabling all other (community) plugins, and disabled hardware acceleration, and it still doesn't work. Any other ideas? I love this plugin...

[fundon]

@thislooksfun Thanks reporting.

Maybe it's same as #255.

You can try this branch #258

[fundon]

@thislooksfun Update to the latest version(v4.1.2).

[thislooksfun]

Nope. Still doesn't work. I'm going to try re-installing it.
EDIT: Re-installing did nothing. Trying a complete re-install of atom.
EDIT 2: Deleting the ~/.atom folder seemed to do the trick. No idea what I did, but I'll try to figure it out.
EDIT 3: It appears that restoring config.cson caused the error again. Will now try to find the settings that's causing the problem.
EDIT 4: Found it! Setting "preferredLineLength" to "999999999" causes the error.
EDIT 5: In fact, any value >= 16384 causes the error.

[fundon]

Good catch! Thanks @thislooksfun

[thislooksfun]

No problem! Just a fair amount of trial and error. Not like I had anything better to do. :P
That number seems weirdly arbetrary though. I can't think of any reason off the top of my head as to why it would make a difference there. If it was a power of two, maybe, but 16384? Weird.

[fundon]

I'm trying to fix that.

[fundon]

@thislooksfun Can you try this branch? https://github.com/fundon/atom-minimap/tree/fix/soft_wrap_at_preferred_line_length

[fundon]

Fixed in v4.1.3.

[fundon]

If it's still same issue, re-open it.

[thislooksfun]

It appears to be working now. However, I have also found that having the line length that high is a bad idea on it's own. Scrolling gets really slow. Thanks for fixing it though! Keep up the great work!

[abe33]

I should have seen it coming. We'll need to make sure the minimap canvas width never get this wide so that we avoid memory or performance hog.

@fundon If you have some time to write the tests corresponding to the fix so that we don't miss it the next time we touch the codebase.

[fundon]

@abe33 You are right 👍 , sorry. I will. :)