[Geany] Visible whitespace

[Lex Trotman]

2009/8/20 Enrico Tröger <enrico.troeger at uvena.de>:
> On Wed, 19 Aug 2009 11:03:41 +1000, Lex wrote:
>
>>2009/8/19 Enrico Tröger <enrico.troeger at uvena.de>:
>>> On Tue, 18 Aug 2009 16:37:43 +1000, Lex wrote:
>>>
>>>>Hi Lee,
>>>>
>>>>2009/8/18 Lee Gold <leegold at operamail.com>:
>>>>>
>>>>> I tried changing colors in files.common. Changing just color does
>>>>> not solve the problem because the symbols are very tiny pin pricks.
>>>>> Changing them to black or red does not help. Changing background
>>>>> color from white totally disrupts the "look".
>>>>
>>>>Unfortunately AFAICT the visible whitespace display is hard coded in
>>>>Scintilla, the editing component used by Geany
>>>>(http://www.scintilla.org).  Geany can only turn it on/off.
>>>>The best that can be done is to fiddle with the settings to minimise
>>>>the disruption whilst maximising visibility.
>>>
>>> Another, in the long term more promising thing to be done is to
>>> request this feature enhancement to the Scintilla project, once it
>>> is available there, it will also be available in Geany (ok, maybe
>>> with some little delay :D).
>>> I don't know whether it's a coincidence or whether it was one
>>> of you but today exactly this request was made:
>>>
>>> https://sourceforge.net/tracker/?func=detail&atid=352439&aid=2839427&group_id=2439
>>
>>I had a look at what was proposed above, it is probably the right way
>>to do it.
>>
>>So I did it the wrong way ;-)
>>
>>To Neils eternal shame :-), the size of the dot is manifest constants
>>in the code, the patch changes it to the symbol
>>SCI_VISIBLE_WHITESPACE_SIZE with a default of one.
>>
>>To change it "make CXXFLAGS=-DSCI_VISIBLE_WHITESPACE_SIZE=2" or
>>whatever size or configure it or however waf does these things.
>>
>>Patch is against scintilla in Geany.
>
> Nice!
> I changed the value to 3 for testing and it gets pretty visible then
> however I personally was also happy with the default but I see that it
> might be too small/bright in some setups.
>
> About the code:
> I'd like to see this code to be discussed with Neil, ideally in the
> already existing feature request in the tracker. I think he would
> accept it and so we don't need to patch our Scintilla copy again.
> Ideally, the value could be changed with a message and not only by
> changing constants in the source code,

True, thats why I said I'd done it the wrong way.  But it only took
half an hour, I don't have the time or knowledge to add a message to
Scintilla.

Will send it to Neil.

Cheers
Lex

 I guess users would appreciate
> that. I guess best would be to ask Neil before doing any of such work.
>
>
> Regards,
> Enrico
>
> --
> Get my GPG key from http://www.uvena.de/pub.asc
>
> _______________________________________________
> Geany mailing list
> Geany at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
>
>