Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parinfer sporadically causes window maximization when opening file requiring changes #98

Open
ghost opened this issue Feb 2, 2018 · 6 comments
Assignees

Comments

@ghost
Copy link

ghost commented Feb 2, 2018

Sometimes when opening a clojure file in Atom that does not conform to the parinfer standards, while the dialog "parinfer needs to change this file", Atom will maximize the window. This is supremely annoying. It doesn't happen at other times.

I'm still figuring out how to reproduce this exactly, and I've only noticed this when dealing with co-workers' source who don't use Parinfer, so I can't provide a "known triggering file". I'm wondering if you've seen/heard of this bug before.

@oakmac
Copy link
Owner

oakmac commented Feb 2, 2018

Thanks for reporting. I have not experienced this bug, nor heard any other reports of it. But sounds like something worth investigating.

Have you isolated this down to atom-parinfer specifically? Could it be another plugin causing this behavior?

@oakmac oakmac self-assigned this Feb 2, 2018
@ghost
Copy link
Author

ghost commented Feb 5, 2018

I wasn't 100% sure it was parinfer, but it was only triggering alongside the parinfer dialog prompting to fix a poorly-formatted file. I also could not reliably reproduce the problem even with the same exact file contents across different Atom sessions.

Anyway, I haven't been able to reproduce the problem since upgrading to 1.22, so I'm going to mark it as closed for now. If I see the problem again I'll try to get more info.

@ghost ghost closed this as completed Feb 5, 2018
@ghost
Copy link
Author

ghost commented Feb 6, 2018

I guess I spoke too soon. It's back. And I can somewhat reliably reproduce, and have verified that it happens only when parinfer is enabled, and will happen when only parinfer is enabled.

The source file in question is a vase-api-descriptor.edn from one of our projects that I can't share, but I'll see if I can't narrow it down to something in particular with the file that causes the problem sometime soon.

@ghost ghost reopened this Feb 6, 2018
@ghost
Copy link
Author

ghost commented Mar 6, 2018

OK, I have an example that, at least right now, with a bare Atom except for Parinfer, is causing Atom to maximize its window every time I open the file.

This is at /env/dev/clj/user.clj in the repository, with the contents:

(ns user
  (:require [mount.core :as mount]

(defn start []
  (mount/start)
)

(defn stop []
  (mount/stop)
)

(defn restart [])

Yes the file is missing an unbalanced paren, and yes the parens that are in there are hanging.

@oakmac
Copy link
Owner

oakmac commented Mar 27, 2018

What platform and version of Atom are you using?

When you open that file, do you see a message like the following:

parinfer-warning

I am trying to recreate this bug, but not having any luck ;)

@oakmac
Copy link
Owner

oakmac commented Aug 3, 2018

I experienced this bug today! Or at least a similar bug.

Steps:

  1. On Mac OSX. Atom v1.24.1
  2. Window was maximized
  3. Opened a file that Parinfer needed to adjust the parens (shown the dialog)
  4. Window went from maximized to a smaller, fixed size.

My hunch is this is something involving the Atom API. Not this extension. But still worth investigating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant