Ink 0.4.0 messes with Atom window size about atom-ink HOT 5 CLOSED

I also ran into this issue. Atom no longer saves the window size and position on my Mac (El Capitan) upon closing. When I disable the Ink package, everything works as expected.

from atom-ink.

This is due to those lines -- no clue why this should change anything about the window size though.

Seems a bit like a bug in Atom: Similar calls involving editors, e.g. atom.workspace.getTextEditors(), change the window size as well when called in the activate function.

Possibly related: atom/atom#9664

from atom-ink.

Those lines have been there for a while, but perhaps they're interacting badly with the new deserialisation code? Either way, editors actually already have IDs, so we should just replace that code.

from atom-ink.

IIRC this issue has been around for quite a while as well (at least for me, but I just never attributed it to ink).

from atom-ink.

Yeah, I don't quite know why I thought those lines were the culprit -- can't reproduce the behaviour I saw when testing a few days back.

That said, I'm pretty sure this is because of those lines now -- what's the failure mode this is supposed to address, @MikeInnes? I don't see any (de)serialisation problems with the console or whatever when commenting this out, but of course I don't know what to look for anyways...

from atom-ink.