[PATCH] wire up visible_{tabs, tws} in editor's config dialog (fix bug 1384)

Denys Vlasenko vda.linux at googlemail.com
Wed Jul 1 13:32:14 UTC 2009


>From bug 1384 description:

=====================================================
I like a new feature of specially displayed tabs
and trailing spaces, but sometimes they are getting
in the way. For example, when I want to cut-and-paste
with the mouse, I usually do not want it.

Also, inevitably there will be users which actively
dislike this feature. It's better for the project
if it supports those users too.

The most comprehensive solution would be two-fold:

Add a config option for it in F9->Options->General
(takes care of those users who hate whitespace
highlighting and want it off by default), and

Add a menu item to F9->Edit to toggle whitespace
highlighting temporarily with a key combination.
=====================================================

I discovered that this is already conditionalzed in the code,
just not tweakable by config dialog.

6.patch fixes it, and also it makes _trailing_ tabs
to show up as tabs if visible_tabs = 0 but visible_tws = 1.

Please apply.

7.patch is follow-up whitespace and formatting cleanup,
no code changes. Apply it if it looks ok to you.
It does not relate to the fix per se, just catering to my
perfectionist itch.

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
--
vda
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6.patch
Type: text/x-patch
Size: 7595 bytes
Desc: not available
URL: <http://lists.midnight-commander.org/pipermail/mc-devel/attachments/20090701/f8116c82/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 7.patch
Type: text/x-patch
Size: 2296 bytes
Desc: not available
URL: <http://lists.midnight-commander.org/pipermail/mc-devel/attachments/20090701/f8116c82/attachment-0001.bin>


More information about the mc-devel mailing list