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

add symbol for noeol #462

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shiluotang
Copy link

@shiluotang shiluotang commented Sep 17, 2024

only show this symbol when g:WebDevIconsNoEndOfLineDefaultSymbol is configured

Requirements (please check off with 'x')

What does this Pull Request (PR) do?

show symbol for noeol status as the airline [!EOL]

How should this be manually tested?

when g:WebDevIconsNoEndOfLineDefaultSymbol is default configured

  1. open a simple file in vim
  2. change eof status by vim command ":set noendofline"
  3. there should be no symbol for noeof on airline fileformat part

when g:WebDevIconsNoEndOfLineDefaultSymbol is configured by user

buffer verify

  1. set g:WebDevIconsNoEndOfLineDefaultSymbol in vimrc
  2. open a simple file in vim
  3. change eof status by vim command ":set noendofline"
  4. airline fileformat part should contain the value of "g:WebDevIconsNoEndOfLineDefaultSymbol"

file verify

  1. set g:WebDevIconsNoEndOfLineDefaultSymbol in vimrc
  2. create a simple file: echo "hello" > abc
  3. open file in binary mode: vim -b abc
  4. call vim command ":set noendofline", then save and exit
  5. open file in normal mode: vim abc
  6. airline fileformat part should contain the value of "g:WebDevIconsNoEndOfLineDefaultSymbol"

Any background context you can provide?

What are the relevant tickets (if any)?

Screenshots (if appropriate or helpful)

Untitled

only show this symbol when g:WebDevIconsNoEndOfLineDefaultSymbol is
configured
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.

1 participant