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

history-substring-search should use zstyle #52

Closed
sorin-ionescu opened this issue Mar 16, 2012 · 11 comments
Closed

history-substring-search should use zstyle #52

sorin-ionescu opened this issue Mar 16, 2012 · 11 comments

Comments

@sorin-ionescu
Copy link
Owner

Since history-substring-search has diverged a little bit from upstream, we might as well convert it fully.

@meh
Copy link
Contributor

meh commented Oct 3, 2012

Umh, can we at least have a way to set the colors from ~/.zpreztorc?

If you set HISTORY_SUBSTRING_SEARCH_HIGHLIGHT_{FOUND,NOT_FOUND} in ~/.zshenv or ~/.zpreztorc it gets overwritten and putting it in ~/.zshrc looks very bad to me.

@ColinHebert
Copy link
Contributor

I guess it would be easy to add a few zstyles to set the colors on {FOUND,NOT_FOUND}.

@lucy
Copy link
Contributor

lucy commented Oct 3, 2012

I think you should set it in ~/.zshrc, since AFAIK both ~/.zshenv and ~/.zpreztorc are sourced before the module is loaded.

@meh
Copy link
Contributor

meh commented Oct 3, 2012

I think you should set it in ~.zshrc, since AFAIK both ~/.zshenv and ~/.zpreztorc are sourced before the module is loaded.

That's why I'm asking to add a zstyle to set the colors, to keep it
consistent.

@ColinHebert
Copy link
Contributor

@neeee Yeah, but everything that is loaded in prezto should be configurable in zpreztorc

@sorin-ionescu
Copy link
Owner Author

I would leave it alone since syntax highlighting is so buggy.

@meh
Copy link
Contributor

meh commented Oct 3, 2012

It's included as module, it's two lines of code to do the setting from the zstyle.

It works perfectly here except for this configuration issue, what's the point of including it if you don't even make it configurable properly?

You keep forgetting this is a community project and not your little personal configuration thing and should be a bit more open minded about it.

If this is not supposed to be a community project just say it clearly so I can get out of here and do something myself.

@sorin-ionescu
Copy link
Owner Author

Are you still on medication? Your attitude still sucks.

Community does not mean that you scream and whine at me until a change is done in your favour. It means that you do it yourself. So, open a pull request. Then, I'll have a look at it.

@meh
Copy link
Contributor

meh commented Oct 3, 2012

Are you still on medication? Your attitude still sucks.

Sure. You should too, your attitude sucks as well.

Community does not mean that you scream and whine at me until a change is done in your favour. It means that you do it yourself. So, open a pull request. Then, I'll have a look at it.

There's a difference between saying "you can do it" and "leave it
alone".

And the former surely isn't what you said.

I asked to have it done because it's two lines that are easier for you
to write than for me to open a pull request and have you whine at the
style and whatnot until you accept it and rewrite it all as you like
it.

Thought skipping the pointless steps would be better for two lines.

@sorin-ionescu
Copy link
Owner Author

I could easily follow the following protocol: If the pull requester can't be bothered to read the contribution rules, and the source code to get a feel for the style, I can't be bothered to read the pull request.

I could just close pull requests that don't conform to the project. Instead, I give people an opportunity to fix them.

If you want an everything goes crap fest, use Oh My Zsh.

Yet, you don't.

You can't have it both ways.

So open your pull request, or forever hold your silence.

@sorin-ionescu sorin-ionescu reopened this Oct 4, 2012
sorin-ionescu added a commit that referenced this issue Dec 23, 2012

Verified

This commit was signed with the committer’s verified signature.
jcubic Jakub T. Jankiewicz
@sorin-ionescu
Copy link
Owner Author

Is something like a833a9a what you wanted?

gmaghera added a commit to gmaghera/prezto that referenced this issue May 1, 2013
Merge remote-tracking branch 'upstream/master'

* upstream/master:
  [Fix sorin-ionescu#419] Rewrite module gpg-agent; rename it to gpg
  [Fix sorin-ionescu#52] Add zstyles to configure history-substring-search
  [sorin-ionescu#52] Add zstyles to configure syntax-highlighting
  Set WORDCHARS in the editor module
  [Fix sorin-ionescu#422] Revert "Remove no longer necessary key bindings"
  Update external history-substring-search
  Sort Git conflict aliases
  [Fix sorin-ionescu#403] Display accented characters properly
  Correct typos in git module README
  Wrap errant long line in git module README
  Fix misspellings of the word bellow in READMEs
  Remove no longer necessary key bindings
  Update external history-substring-search
  Update external syntax-highlighting
  Update external completions
  [Fix sorin-ionescu#399] Only enable the main syntax highlighter
  Use the builtin $signals array
admk added a commit to admk/prezto that referenced this issue May 20, 2013
… HEAD

* 'master' of https://github.com/sorin-ionescu/prezto: (35 commits)
  Make gpg-agent and ssh-agent work with each other
  [Fix sorin-ionescu#425] Rewrite module ssh-agent; rename it to ssh
  [Fix sorin-ionescu#103] Add documentation for editor
  Remove the git-info SIGINT message
  [Fix sorin-ionescu#307] Do not auto-off git-info
  Remove ununsed variable
  Clarify Git listing aliases descriptions
  Swap aliases gsd and gsL
  Rename alias gRc to gRp
  [Fix sorin-ionescu#221] Add a simple git-info
  [sorin-ionescu#221] Do not format undefined zstyles
  Initialize ahead and behind local variables
  Add rar command to archive module
  Refactor Emacs module
  Load completion for Carton
  Correct syntax error in variable assignment
  Ensure that the tmux server is started
  [Fix sorin-ionescu#426] Correct syntax error in variable assignment
  [Fix sorin-ionescu#419] Rewrite module gpg-agent; rename it to gpg
  [Fix sorin-ionescu#52] Add zstyles to configure history-substring-search
  ...
jeffknupp pushed a commit to jeffknupp/prezto that referenced this issue Oct 15, 2013
linuslundahl pushed a commit to linuslundahl/prezto that referenced this issue Oct 17, 2013
zeroasterisk pushed a commit to zeroasterisk/prezto that referenced this issue Oct 22, 2013
lildude pushed a commit to lildude/prezto that referenced this issue Jan 12, 2014
matthoffman pushed a commit to matthoffman/oh-my-zsh that referenced this issue Sep 18, 2014
matthoffman pushed a commit to matthoffman/oh-my-zsh that referenced this issue Sep 18, 2014
rooney pushed a commit to rooney/prezto that referenced this issue Aug 19, 2020
RIT80 pushed a commit to RIT80/prezto that referenced this issue Jan 25, 2022
RIT80 pushed a commit to RIT80/prezto that referenced this issue Jan 25, 2022
RIT80 added a commit to RIT80/prezto that referenced this issue Jan 25, 2022
RIT80 added a commit to RIT80/prezto that referenced this issue Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants