Discussion:
New mode to view whitespace within selected text
Farshid Lashkari
2013-07-30 20:20:48 UTC
Permalink
Hi,

I've attached a patch that adds a new SCI_SETVIEWWS mode
called SCWS_VISIBLEINSELECTION. The new mode enables viewing of whitespace
only within selected text.

Cheers,
Farshid
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Farshid Lashkari
2013-07-30 21:58:46 UTC
Permalink
Please use the attached patch instead. There was a problem with the
previous attachment.
Post by Farshid Lashkari
Hi,
I've attached a patch that adds a new SCI_SETVIEWWS mode
called SCWS_VISIBLEINSELECTION. The new mode enables viewing of whitespace
only within selected text.
Cheers,
Farshid
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Neil Hodgson
2013-07-31 01:20:54 UTC
Permalink
Please use the attached patch instead. There was a problem with the previous attachment.
The changed files appear to be based on 3.3.0 and those files have changed enough that it is difficult to apply to the current release 3.3.4. It would be helpful to send changed files based on 3.3.4 or patches against 3.3.4 or the Hg repository.

Neil
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Farshid Lashkari
2013-07-31 03:50:59 UTC
Permalink
Sorry about that Neil. Here are the changes applied to 3.3.4. In the
future, do you prefer whole files or diffs?
Post by Farshid Lashkari
Post by Farshid Lashkari
Please use the attached patch instead. There was a problem with the
previous attachment.
The changed files appear to be based on 3.3.0 and those files have
changed enough that it is difficult to apply to the current release 3.3.4.
It would be helpful to send changed files based on 3.3.4 or patches against
3.3.4 or the Hg repository.
Neil
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Neil Hodgson
2013-07-31 23:53:22 UTC
Permalink
Got this working but it doesn't appear useful to me. Does anyone else think this is a good idea?

Neil
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Matthew Brush
2013-08-01 00:15:22 UTC
Permalink
Post by Neil Hodgson
Got this working but it doesn't appear useful to me. Does anyone else think this is a good idea?
I don't think it's a bad idea :)

Would have to try it out for a bit to be sure. Maybe it could be
"provisional" (pending anyone saying it's great or sucks)?

Cheers,
Matthew Brush
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Philippe Lhoste
2013-08-01 07:40:03 UTC
Permalink
Post by Neil Hodgson
Got this working but it doesn't appear useful to me. Does anyone else think this is a good idea?
I admit I fail to see the use case for this feature.
I just use, in SciTE, the shortcut to toggle the whitespace display, allowing me to
quickly see it when I need it, and to hide it as quickly. No (personal) need to limit this
view to the selection.
--
Philippe Lhoste
-- (near) Paris -- France
-- http://Phi.Lho.free.fr
-- -- -- -- -- -- -- -- -- -- -- -- -- --
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Lex Trotman
2013-08-01 08:29:03 UTC
Permalink
Post by Philippe Lhoste
Post by Neil Hodgson
Got this working but it doesn't appear useful to me. Does anyone else
think this is a good idea?
I admit I fail to see the use case for this feature.
I just use, in SciTE, the shortcut to toggle the whitespace display,
allowing me to quickly see it when I need it, and to hide it as quickly. No
(personal) need to limit this view to the selection.
Most (all?) Scintilla based applications provide for toggling visible
whitespace with a shortcut key so this does not seem to be very useful.
Perhaps the original proposer should be the one to present its use-case.

Cheers
Lex
Post by Philippe Lhoste
--
Philippe Lhoste
-- (near) Paris -- France
-- http://Phi.Lho.free.fr
-- -- -- -- -- -- -- -- -- -- -- -- -- --
--
You received this message because you are subscribed to the Google Groups
"scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an
.
.
Visit this group at http://groups.google.com/**group/scintilla-interest<http://groups.google.com/group/scintilla-interest>
.
For more options, visit https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/opt_out>
.
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Farshid Lashkari
2013-08-01 18:32:55 UTC
Permalink
I also use a shortcut to toggle viewing of whitespace as needed, since I
don't like it to be always visible. However, I was noticing that in many of
these cases, the region I wanted to view the whitespace was already within
a selection. I thought this could reduce the number of times I needed to
toggle the whitespace. I admit it's not a huge time saver, but convenient
in some cases. If it's not worth the extra code complexity and maintenance
effort, then I'm fine with not including it.
Post by Farshid Lashkari
Hi,
I've attached a patch that adds a new SCI_SETVIEWWS mode
called SCWS_VISIBLEINSELECTION. The new mode enables viewing of whitespace
only within selected text.
Cheers,
Farshid
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Neil Hodgson
2013-08-03 04:21:29 UTC
Permalink
If it's not worth the extra code complexity and maintenance effort, then I'm fine with not including it.
For now, I won't commit it. If more demand appears then it may go in.

The core drawing code is very important. Its been changing recently to implement character representations and I'm working on a fix for a slow-down caused by that feature. I'd like to refactor the code to move drawing out of the Editor class so that it will be easier to improve and reuse.

Neil
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Mike Lischke
2013-08-03 08:12:10 UTC
Permalink
Neil,
Post by Neil Hodgson
If it's not worth the extra code complexity and maintenance effort, then I'm fine with not including it.
For now, I won't commit it. If more demand appears then it may go in.
The core drawing code is very important. Its been changing recently to implement character representations and I'm working on a fix for a slow-down caused by that feature. I'd like to refactor the code to move drawing out of the Editor class so that it will be easier to improve and reuse.
Would it be possible to add support for something similar like text
attachments as in NSTextField?

I'm working on enhancing the code completion feature in our app and
would like to display pseudo input for function parameters and the
like, just like XCode does during code completion. I also use text
attachments in another app to show (custom rendered) tags. Tags have
not much to do with code editing, but it shows another example for a
feature like this.
Post by Neil Hodgson
Neil
Mike
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Neil Hodgson
2013-08-03 13:27:11 UTC
Permalink
Post by Mike Lischke
Would it be possible to add support for something similar like text
attachments as in NSTextField?
I'm working on enhancing the code completion feature in our app and
would like to display pseudo input for function parameters and the
like, just like XCode does during code completion.
That's more like virtual text which has been proposed for Scintilla.
https://sourceforge.net/p/scintilla/feature-requests/969/

Neil
--
You received this message because you are subscribed to the Google Groups "scintilla-interest" group.
To unsubscribe from this group and stop receiving emails from it, send an email to scintilla-interest+***@googlegroups.com.
To post to this group, send email to scintilla-***@googlegroups.com.
Visit this group at http://groups.google.com/group/scintilla-interest.
For more options, visit https://groups.google.com/groups/opt_out.
Loading...