Skip to content

Fix balloon API on GUI #1064

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

Merged
merged 2 commits into from
Mar 8, 2021
Merged

Fix balloon API on GUI #1064

merged 2 commits into from
Mar 8, 2021

Conversation

ichizok
Copy link
Member

@ichizok ichizok commented Jul 29, 2020

  • Fix the position of calling setLastToolTip: in gui_mch_retain_font()
  • Set balloonEval->msg in gui_mch_post_balloon()

This fix makes Test_balloon_show_gui pass.

@ychin
Copy link
Member

ychin commented Aug 11, 2020

I haven't looked at this change much yet. @ichizok does this improve the functionality or just fix the test?

@ichizok ichizok force-pushed the fix/balloon-gui branch 2 times, most recently from fcc6d5f to ac4b1d1 Compare September 28, 2020 07:24
@ychin ychin added this to the snapshot-168 milestone Dec 12, 2020
@ichizok ichizok force-pushed the fix/balloon-gui branch 2 times, most recently from 9fea408 to df5bb8b Compare December 20, 2020 14:02
@ychin
Copy link
Member

ychin commented Jan 4, 2021

Hi @ichizok is this PR ready? I asked a couple questions and just wondering if you took a look at them. I think the PR is mostly fine and I'm not sure if there are ways around the private API call but just wondered what your thoughts were.

@ichizok
Copy link
Member Author

ichizok commented Jan 5, 2021

I'm sorry for the late reply, this is ready to merge.

does this improve the functionality or just fix the test?

This is the fix for balloon_show() on GUI and enable the test about it.

* Fix the position of calling setLastToolTip: in gui_mch_retain_font()
* Set balloonEval->msg in gui_mch_post_balloon()

This fix makes Test_balloon_show_gui pass.
@ychin ychin merged commit 58064dc into macvim-dev:master Mar 8, 2021
@ichizok ichizok deleted the fix/balloon-gui branch March 8, 2021 08:00
ychin added a commit that referenced this pull request Mar 8, 2021
Updated to Vim 8.2.2576.

Features
====================

Apple Silicon Support
--------------------

MacVim's binary release now fully supports Apple Silicon! If you have an
M1 Mac you should notice MacVim being more snappy and smoother. #1150

If you would like to (e.g. if you only installed Python 3 in Rosetta and
use plugins that use Python), you could run MacVim under Rosetta. You
could use `:version` while in MacVim to tell whether you are running in
Rosetta / Intel or Apple Silicon by seeing whether it says x86_64 or
arm64. (Vim 8.2.2174)

If you rely on Python/Ruby/Lua integration, note that previously MacVim
by default searches the `/usr/local/` path for installed language
runtimes. With this release, MacVim will still search those folders
under Intel / x86-64 builds, but under Apple Silicon / M1, MacVim will
search under `/opt/homebrew/` instead, which is the default folder for
Homebrew under Apple Silicon. If you don't use Homebrew, or installed
language runtimes under other folders, you would need to set
`python3dll`/`luadll`/`rubydll` in your vimrc.

Full Screen Fixes and Improvements
--------------------

Removed the fade-to-black animation when transition to full screen, as
they were distracting and looked jarring. For non-native full screen,
you could set `MMFullScreenFadeTime` to a non-zero value to still get
the animation back. #1171

Non-native full screen

- Non-native full screen now has an option to show menu bar when it's
  active (under Appearance preference pane). #1170

- Fixed non-native full screen to properly hide the menu / dock when
  used on a secondary screen. Also, fixed misc issues with non-native
  full screen not drawing at the right offset. #7 #1155 #1170

- Fixed non-native full screen's `fuopt` setting. It now works again.
  This feature allows you to limit only expand horizontally or
  vertically when using non-native full screen to help focus on the
  content, see `:help fuopt`. #509

Fixed small bug in Touch Bar's full screen button sometimes not being
updated correctly. #1171

Known Issues
====================

Text invisible after plugging in monitor or waking from sleep
--------------------

There is currently a known issue in rendering where after
plugging/unplugging an external monitor, or waking from sleep when
connected to a monitor, there is a small change MacVim will stop drawing
text. If you see that, please report that to #1164. This release
contains some additional logging to help dignose the issue and we are
still looking to root cause it.

General
====================

- Added an option to ignore font's line height. r168 introduced a new
  renderer which changed the behavior for how font's line height works
  -- instead of ignoring it, the new renderer respects the line height
  of the font. This new setting allows MacVim to behave in the old way,
  since some fonts have large line height that the user may not want to
  use. #1152

Fixes
====================

- Fixed balloon APIs (`balloon_show()` / `balloon_gettext()`) so plugins
  relying on them should now work. #902 #1064

Misc
====================

- Scripting languages versions:
    - Ruby is now built against 3.0, up from 2.7.

Compatibility
====================

Requires macOS 10.9 or above.

Script interfaces have compatibility with these versions:

- Lua 5.4
- Perl 5.18
- Python2 2.7
- Python3 3.9
- Ruby 3.0
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 this pull request may close these issues.

2 participants