Ticket #1365 (closed enhancement: fixed)

Opened 10 years ago

Last modified 6 years ago

fancy highlighted tabs and spaces even while F3 block marking

Reported by: ossi Owned by:
Priority: major Milestone: 4.7
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description

due to the lack of any reaction to the comment i made, i'm now filing a complementary ticket for ticket #342, i.e., i'm asking for a complete revert of the change.

disabling whitespace visualization in selections was intentional. not doing so breaks two things at once:

  • due to the uniform coloring of the selection, the visible whitespace is in no way subtle any more. consequently selected text with lots of whitespace (say, properly tabbed out source code) becomes something between plain ugly and totally unreadable.
  • the workaround to make text usefully selectable for mouse-based copy&paste in x is gone, as there is now no way to get rid of the whitespace vizualizers. this was supposed to be quickly toggleable at some point, but it still doesn't seem to be, so the workaround is crucial.

and the obvious question is: why on earth do you want visible whitespace in selections? trailing whitespace is visualized by the selection itself, and i see no use case for the distinction between tabs and spaces in this context.

Change History

comment:1 Changed 10 years ago by ossi

  • Component changed from mc-core to mcedit

comment:2 follow-up: ↓ 4 Changed 10 years ago by angel_il

I want to see that text which sesected.
If you wish not to see tabs when you select by Shift-mouse in X-clipboard switch off "show tabs". Hotkey for this will be in 4.7 version

comment:3 Changed 10 years ago by angel_il

  • Type changed from defect to enhancement

it's not bug.

comment:4 in reply to: ↑ 2 ; follow-up: ↓ 6 Changed 10 years ago by ossi

Replying to angel_il:

I want to see th[e] text which [is] se[l]ected.

ehh ... you do in any case. the one thing you don't see is whether some whitespace are tabs or spaces. why do you need that while doing a selection? what would be a concrete use case for it?

comment:5 Changed 10 years ago by angel_il

Replying to ossi:

Replying to angel_il:

I want to see th[e] text which [is] se[l]ected.

ehh ... you do in any case. the one thing you don't see is whether some whitespace are tabs or spaces. why do you need that while doing a selection? what would be a concrete use case for it?

ok, why you need see any selected chars? i want to see text that i select.
Hotkey for switch off highlight trailing spaces and tab will be so soon.

comment:6 in reply to: ↑ 4 ; follow-up: ↓ 7 Changed 10 years ago by angel_il

Replying to ossi:

Replying to angel_il:

I want to see th[e] text which [is] se[l]ected.

ehh ... you do in any case. the one thing you don't see is whether some whitespace are tabs or spaces. why do you need that while doing a selection? what would be a concrete use case for it?

ok, why you need see any selected chars? i want to see text that i select.
Hotkey for switch off highlight trailing spaces and tab will be so soon.

comment:7 in reply to: ↑ 6 Changed 10 years ago by ossi

  • severity set to ---

Replying to angel_il:

ok, why you need see any selected chars? i want to see text that i select.

for orientation, obviously. but when the whitespace visualisation cannot be distinguished from normal text, it becomes not only ugly, but also confusing.

comment:8 Changed 10 years ago by angel_il

  • Status changed from new to closed
  • Resolution set to fixed

see #1384

comment:9 Changed 6 years ago by ossi

  • Branch state set to no branch

the original problem is still there, and the hotkey for the workaround was never added (which wouldn't be a proper solution anyway).
the problem that this interferes with the selection is now minor, as the xclip "integration" makes PRIMARY selection mostly irrelevant for copying. however, it still looks massively ugly without adding any benefit.
therefore i think this should be re-opened.

Note: See TracTickets for help on using tickets.