Giter VIP home page Giter VIP logo

Comments (12)

tpope avatar tpope commented on July 21, 2024

Did you see my commit where I explained how much this helps with Press ENTER prompts?

This is actually one of the ones I expected might be controversial. I'm quite willing to consider yanking it. But let's give it a few days to see what everyone else has to say.

I'd like to make everything user overridable, but it's hard (though not impossible) to tell the difference between a Vim default and a user setting. As a workaround, you could most certainly override it in .vim/after/plugin/sensible.vim.

from vim-sensible.

teoljungberg avatar teoljungberg commented on July 21, 2024

I'd like to make everything user overridable, but it's hard (though not impossible) to tell the difference between a Vim default and a user setting. As a workaround, you could most certainly override it in .vim/after/plugin/sensible.vim

That's rather unpleasant! For this to actually be sensible-settings every setting should able to be overridden. We don't want a handful of forks, each with their own modifications

from vim-sensible.

ku1ik avatar ku1ik commented on July 21, 2024

As for overriding things, I have a thought: maybe the "README recommended" way of loading this "plugin" should be "download + manually source at the top of .vimrc" rather than "use plugin manager and then override in .vim/after/*" ?

from vim-sensible.

tpope avatar tpope commented on July 21, 2024

I would hope most people wouldn't respond to a minor inconvenience in overriding with a fork. Seems like a petty way to handle it.

Why the aversion to after/plugin/sensible.vim? If you really, really must have everything in your vimrc, you can do

autocmd VimEnter * set cmdheight=1

I'm going to wait for a second option to be objected to before I worry about this any more. If it's really just cmdheight, I'll nuke it and call it a day.

from vim-sensible.

tpope avatar tpope commented on July 21, 2024

@sickill my goal is to minimize vimrc junk, and this sounds like a step in the wrong direction. But you most certainly could use it that way if you wanted to

runtime! plugin/sensible.vim
set cmdheight=1

The include guard will prevent sensible.vim from loading a second time.

from vim-sensible.

sheerun avatar sheerun commented on July 21, 2024

Is there a way to increase cmdheight temporally, to match numer of displayed error lines?

from vim-sensible.

tpope avatar tpope commented on July 21, 2024

I don't think so. There's no autocommand for an error occurring. Even trickier would be deciding when to shrink it back down.

Is this a vote against cmdheight=2?

from vim-sensible.

sheerun avatar sheerun commented on July 21, 2024

For me these two lines are not esthetically pleasing, unfortunately ;/

If there is no hook, maybe there exist a way to display only first message in most annoying cases. For example echo Hello displays following ones:

E121: Undefined variable: Hello
E15: Invalid expression: Hello

But are these two really necessary?

I also can't find your commit with explanation, so sorry if I'm repeating something.

In principle this repository should be:

a universal set of defaults that (hopefully) everyone can agree on

And I guess not everyone is happy with that (as you've expected).

So probably it doesn't matter if it's right or wrong, it's just the default.

from vim-sensible.

tpope avatar tpope commented on July 21, 2024

Removed. Maybe I'll make a different plugin for "options tpope thinks you should set."

@sheerun, Vim's error messages are what they are. There's not really much we can do about them.

from vim-sensible.

teoljungberg avatar teoljungberg commented on July 21, 2024

The most sensible way out of this issue is to allow every setting to be easilly overridden.
The default doesn't really mind me, aslong as I can override those options I don't agree on I'm perfectly happy!

from vim-sensible.

tpope avatar tpope commented on July 21, 2024

I gave 3 different ways to override. Are none of them "easy" enough for you?

from vim-sensible.

rizzatti avatar rizzatti commented on July 21, 2024

@tpope why not have a g:sensible_tpope_options for that "options tpope thinks you should set". Would be funny at least ;)

from vim-sensible.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.