-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix #6605: Remove soft-wrap.enable option wrapping. #6656
Conversation
(Had to revert in master, @pascalkuthe pointed out a subtle bug with config override handling) |
This is not the right fix for the problem. Softwrap can be configured both per language and globally. The language configuration takes precedence over the global config. Therefore all softwrap options are options so that we can fallback to the global config. With this PR the following language config would disable softwrap for [[language]]
name ="markdown"
softwrap.wrap-at-text-width = true In this case there is a language specific To fix #6605 twe should either add a spegifal case to the default implementation of |
…x-editor#6656) Co-authored-by: gibbz00 <gabrielhansson@gmail.com>
…g. (helix-editor#6656)" This caused a bug that would ignore the global config. This reverts commit af88a3c.
…x-editor#6656) Co-authored-by: gibbz00 <gabrielhansson@gmail.com>
…g. (helix-editor#6656)" This caused a bug that would ignore the global config. This reverts commit af88a3c.
…x-editor#6656) Co-authored-by: gibbz00 <gabrielhansson@gmail.com>
…g. (helix-editor#6656)" This caused a bug that would ignore the global config. This reverts commit af88a3c.
No description provided.