Skip to content

New Window/Drag-Drop currently open file fails #913

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

Closed
Paulv445 opened this issue Jun 17, 2019 · 4 comments · Fixed by #919
Closed

New Window/Drag-Drop currently open file fails #913

Paulv445 opened this issue Jun 17, 2019 · 4 comments · Fixed by #919
Milestone

Comments

@Paulv445
Copy link

Custom Version 8.1.1517 (156)
macOX Mojave Version 10.14.5 (18F132)
File opened by drag-drop to MacVim App.
MacVim/right-mouse, New Window
Drag-Drop same file to the new window.
Until this release it opened read only.
With this release hangs, with MacVim quit required to kill window.

@eirnym
Copy link
Contributor

eirnym commented Jun 17, 2019

I have macOS Mojave Version 10.14.6 (18G48f)

the same workflow makes the second window unusable, but if close the opened file and/or open, it could be opened again. Looks like dialog window should be shown, but it either invisible or hidden by a window

@Paulv445
Copy link
Author

Paulv445 commented Jun 17, 2019 via email

@ychin ychin added this to the snapshot-157 milestone Jun 21, 2019
@ychin
Copy link
Member

ychin commented Jun 21, 2019

I can reproduce this. This is happening due to #895, and only happens when swap files are enabled. This should be fixed next release.

ychin added a commit to ychin/macvim that referenced this issue Jul 7, 2019
MMBackend's `handleOpenWithArguments` previously set a "flushDisabled"
flag to disable all flushing during handling the files. This was unsafe,
because under the new code that handles editing immediately (instead of
building a deferred set of Ex commands) MacVim tries to immediately pop
up a dialog box but that message doesn't get properly flushed because
it's disabled. Just remove the setting of that disabled flag as it
doesn't seem like it's gaining anything.

If we want to have a way to disable flushing for performance reasons in
the future , we should make sure the "force" flag in flushQueue:
actually gets respected and used properly (only when we want to force
it). Right now the "force" flag isn't actually used.

Also, make sure the dialog handling code handles the "no GUI resize"
resize message (which gets set when we have `guioptions+=k`) as well and
don't drop the message. Otherwise if drag-and-drop opens a new tab
(since there are multiple files dropped) and the user has guioptions+=k,
it won't get redrawn properly because the
SetTextDimensionsNoResizeWindowMsgID message would get dropped. It's
unfortunate it's a hardcoded hack like this and this should be revisited
in the future.

Fix macvim-dev#913
@ychin ychin closed this as completed in #919 Jul 7, 2019
ychin added a commit to ychin/macvim that referenced this issue Jul 21, 2019
Vim patch 8.1.1722

Features:

- Add 'MMDisableLaunchAnimation' option to disable the opening animation
  of MacVim Windows. macvim-dev#915
- Make 'MMTitlebarAppearsTransparent' work in pre-Mojave macOS versions.
  macvim-dev#917

Fixes:

- Fix issue where MacVim would freeze when opening a file already opened
  in another MacVim window when swap files are enabled. macvim-dev#913
- Menu items with name starting with ']' now properly get hidden in the
  main menu bar (see `:help hidden-menus`). macvim-dev#853

Targets macOS 10.8+

Script interfaces have compatibility with these versions:

- Lua 5.3
- Perl 5.18
- Python2 2.7
- Python3 3.7
- Ruby 2.6
ychin added a commit that referenced this issue Jul 28, 2019
Vim patch 8.1.1722

Features:

- Add 'MMDisableLaunchAnimation' option to disable the opening animation
  of MacVim Windows. #915
- Make 'MMTitlebarAppearsTransparent' work in pre-Mojave macOS versions.
  #917

Fixes:

- Fix issue where MacVim would freeze when opening a file already opened
  in another MacVim window when swap files are enabled. #913
- Menu items with name starting with ']' now properly get hidden in the
  main menu bar (see `:help hidden-menus`). #853

Targets macOS 10.8+

Script interfaces have compatibility with these versions:

- Lua 5.3
- Perl 5.18
- Python2 2.7
- Python3 3.7
- Ruby 2.6
@Paulv445
Copy link
Author

Paulv445 commented Jul 29, 2019 via email

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

Successfully merging a pull request may close this issue.

3 participants