Add macaction to maximize the window.

[digicannon]

Many Vim users, myself included, prefer to start Vim in a maximized window state. For this, I have added the performMaximize: macaction. This allows the following to be added to a vimrc:

autocmd GUIEnter * macaction performMaximize:

[chdiza]

Is this different from the already-existing performZoom:?

Also, what's wrong with just doing the following?

autocmd GUIEnter * set lines=999 columns=999

[juliuszint]

There is also the possibility to add the following line to your .gvimrc
:set fullscreen

[digicannon]

There is also the possibility to add the following line to your .gvimrc
:set fullscreen

Maximize is not fullscreen.

[digicannon]

Is this different from the already-existing performZoom:?

Also, what's wrong with just doing the following?

autocmd GUIEnter * set lines=999 columns=999

performZoom: usually only enlarges the window to match the height of the screen, not the width. The size it chooses depends on what the user last resized it to.

That autocmd does work 100%, but it feels hackish to me. I'd prefer a more clear command.
maca performMaximize: is a much more simple command to type if your window is already open.

[eirnym]

AFAIK, MacVim preserves last used size if nothing changes it manually.

Returning to your implementation, I'd prefer to make it to restore "last used size if maximised" like it was done in performZoom:.

[ychin]

I'm hesitant to add this because we already have MMZoomBoth that allows you to tell MacVim to default to zoom in both directions to fill the screen. I would imagine most users want MacVim to either always zoom vertically, or in both directions?

What's more concerning is that autocmd GUIEnter * macaction performZoom: doesn't actually work probably because or ordering issues. Does your code using performMaximize: actually work reliably even if you open multiple windows using Cmd-N?

Edit:

Also, we should probably not call it "perform"-something as that's a macOS thing. "performZoom:" means pretend to hit the zoom button. We already have a macaction called zoomAll:. Maybe we just need one called zoom: (or maybe zoomBothDirections: if we really want it) that's a little more robust than performZoom:.