Hello,
Here are 3 points to improve (imo) the search & replace tool in geany.
-1- whole words The current "match whole word only" setting fails if the searched expression is not a word at all. What I would expect is that it requires whole word(s) only for the part(s) of the expression that are words -- according to the definition of word characters in the filetype config. Meaning that looking for "+" would not fail only because "match whole word only" is checked, because this setting is irrelevant for this expression. Searching for " = name" could succeed with this option set as expected, but fail on " = names". Am I clear? The current behaviour leads to constantly uncheck this option when irrelevant, which may be dangerous for next search & replace, each time we forget to set it back.
-2- record settings The common settings we use (for me: escape sequence, case sensitive, whole word only, not close dialog) are not recorded between sessions and cannot be set in preferences. As a consequence, we need to redefine the dialog for each working session -- and again some missing or error may be dangerous. Would it be possibly to add these settings to the "search" frame in preferences?
-3- comments & strings A very common task when improving, extending or refactoring code is renaming -- if only for clarity. Too often we'll have to search & replace words that are "key words" for the application, so that they may happen in comments and even in strings. Depending on the case we want the replacement to extend comments and string -- or not at all! Unnoticed and unintended replacements commonly lead to funny but meaningless comments or unpredictable issues because of string changes. A pair of settings to address this point would be great.
Denis
________________________________
la vita e estrany
Hello,
I have packaged a port of Notepad++'s port of Textmate's Vibrant Ink theme. I've posted it at Google Code:
http://code.google.com/p/geany-vibrant-ink-theme/
Enjoy! Jason
On Tue, 24 Nov 2009 02:34:58 -0500, Jason Willson jason.willson@gmail.com wrote:
Hello,
I have packaged a port of Notepad++'s port of Textmate's Vibrant Ink theme. I've posted it at Google Code:
Cool.
Just a few questions / ideas:
- include a README in the tarball/zip archive to give some general information about the contents - add a screenshot on the project page to easily show the potential users how it looks like - are you sure the MIT license is applicable, you stated you ported it from Notepad++'s theme which is based on Textmate's one. There might be problems because Textmate is non-free AFAIK and Notepad++ is probably licenced as GPLv2 (didn't check it). Though this doesn't necessarily mean the themes are licenced as well under these licences. Just check it.
Once the licence issue is solved, I'll add it to the http://www.geany.org/Download/Extras page.
Regards, Enrico
P.S.: Please don't hi-jack other threads.
On Sat, 2009-11-28 at 12:47 +0100, Enrico Tröger wrote:
On Tue, 24 Nov 2009 02:34:58 -0500, Jason Willson jason.willson@gmail.com wrote:
Hello,
I have packaged a port of Notepad++'s port of Textmate's Vibrant Ink theme. I've posted it at Google Code:
Cool.
Just a few questions / ideas:
- include a README in the tarball/zip archive to give some general
information about the contents
- add a screenshot on the project page to easily show the potential
users how it looks like
- are you sure the MIT license is applicable, you stated you ported it
from Notepad++'s theme which is based on Textmate's one. There might be problems because Textmate is non-free AFAIK and Notepad++ is probably licenced as GPLv2 (didn't check it). Though this doesn't necessarily mean the themes are licenced as well under these licences. Just check it.
Once the licence issue is solved, I'll add it to the http://www.geany.org/Download/Extras page.
Regards, Enrico
P.S.: Please don't hi-jack other threads. _______________________________________________ Geany mailing list Geany@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
Excellent advice.
License Issue: I've emailed the author of the original Vibrant Ink theme for Textmate and asked him for advice regarding the licensing of his theme as well as the author of the port to Notepad++. I've temporarily licensed it under the Artistic License/GPL because that seems the most compatible with the GNU Free License that the port is licensed under.
Screenshot: I've added a screenshot to the Project Home page.
Readme: I'll work on that and have that included in the tarball soon.
Thanks again Enrico. I apologize for hi-jacking another thread - I'm new to mailing lists, so I don't exactly have the etiquette yet. I'll keep my page updated and I can update the list as well. I welcome any further advice.
Regards, Jason
On Sat, 28 Nov 2009 19:09:37 -0500, Jason wrote:
On Sat, 2009-11-28 at 12:47 +0100, Enrico Tröger wrote:
On Tue, 24 Nov 2009 02:34:58 -0500, Jason Willson jason.willson@gmail.com wrote:
Hello,
I have packaged a port of Notepad++'s port of Textmate's Vibrant Ink theme. I've posted it at Google Code:
Cool.
Just a few questions / ideas:
- include a README in the tarball/zip archive to give some general
information about the contents
- add a screenshot on the project page to easily show the potential
users how it looks like
- are you sure the MIT license is applicable, you stated you ported
it from Notepad++'s theme which is based on Textmate's one. There might be problems because Textmate is non-free AFAIK and Notepad++ is probably licenced as GPLv2 (didn't check it). Though this doesn't necessarily mean the themes are licenced as well under these licences. Just check it.
Once the licence issue is solved, I'll add it to the http://www.geany.org/Download/Extras page.
Regards, Enrico
P.S.: Please don't hi-jack other threads. _______________________________________________ Geany mailing list Geany@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
Excellent advice.
License Issue: I've emailed the author of the original Vibrant Ink theme for Textmate and asked him for advice regarding the licensing of his theme as well as the author of the port to Notepad++. I've temporarily licensed it under the Artistic License/GPL because that seems the most compatible with the GNU Free License that the port is licensed under.
Ok.
Screenshot: I've added a screenshot to the Project Home page.
I added it to http://www.geany.org/Download/Extras. Though the image is a bit big for the site. We could resize it using Wiki markup but then the alignment breaks (limitation in the Wiki markup). So it'd be nice if you could alternatively provide a smaller image, maybe scaled to 200px or 250px in width, so I could use that one for the page. If you can't or don't want to, just say so, then I put a smaller version on geany.org but I think it'd be nicer to use "your" image, so if you update it, you don't need me.
Readme: I'll work on that and have that included in the tarball soon.
Cool.
Thanks again Enrico. I apologize for hi-jacking another thread - I'm new to mailing lists, so I don't exactly have the etiquette yet. I'll
Don't worry, just don't do it again :).
Regards, Enrico
On Sun, 2009-11-29 at 21:18 +0100, Enrico Tröger wrote:
On Sat, 28 Nov 2009 19:09:37 -0500, Jason wrote:
On Sat, 2009-11-28 at 12:47 +0100, Enrico Tröger wrote:
On Tue, 24 Nov 2009 02:34:58 -0500, Jason Willson jason.willson@gmail.com wrote:
Hello,
I have packaged a port of Notepad++'s port of Textmate's Vibrant Ink theme. I've posted it at Google Code:
Cool.
Just a few questions / ideas:
- include a README in the tarball/zip archive to give some general
information about the contents
- add a screenshot on the project page to easily show the potential
users how it looks like
- are you sure the MIT license is applicable, you stated you ported
it from Notepad++'s theme which is based on Textmate's one. There might be problems because Textmate is non-free AFAIK and Notepad++ is probably licenced as GPLv2 (didn't check it). Though this doesn't necessarily mean the themes are licenced as well under these licences. Just check it.
Once the licence issue is solved, I'll add it to the http://www.geany.org/Download/Extras page.
Regards, Enrico
P.S.: Please don't hi-jack other threads. _______________________________________________ Geany mailing list Geany@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
Excellent advice.
License Issue: I've emailed the author of the original Vibrant Ink theme for Textmate and asked him for advice regarding the licensing of his theme as well as the author of the port to Notepad++. I've temporarily licensed it under the Artistic License/GPL because that seems the most compatible with the GNU Free License that the port is licensed under.
Ok.
I've included two wiki pages which cite the email conversations I had with the original author and the author who ported to Notepad++:
http://code.google.com/p/geany-vibrant-ink-theme/wiki/tm_theme_permission http://code.google.com/p/geany-vibrant-ink-theme/wiki/npp_port_permission
Screenshot: I've added a screenshot to the Project Home page.
I added it to http://www.geany.org/Download/Extras. Though the image is a bit big for the site. We could resize it using Wiki markup but then the alignment breaks (limitation in the Wiki markup). So it'd be nice if you could alternatively provide a smaller image, maybe scaled to 200px or 250px in width, so I could use that one for the page. If you can't or don't want to, just say so, then I put a smaller version on geany.org but I think it'd be nicer to use "your" image, so if you update it, you don't need me.
I've added a scaled-down version: http://geany-vibrant-ink-theme.googlecode.com/files/vibrant_ink_screenshot_s...
Readme: I'll work on that and have that included in the tarball soon.
Cool.
Done.
Thanks again Enrico. I apologize for hi-jacking another thread - I'm new to mailing lists, so I don't exactly have the etiquette yet. I'll
Don't worry, just don't do it again :).
Regards, Enrico
Geany mailing list Geany@uvena.de http://lists.uvena.de/cgi-bin/mailman/listinfo/geany
Late reply...
On Mon, 23 Nov 2009 19:19:47 +0100 spir denis.spir@free.fr wrote:
Here are 3 points to improve (imo) the search & replace tool in geany.
-1- whole words The current "match whole word only" setting fails if the searched expression is not a word at all. What I would expect is that it requires whole word(s) only for the part(s) of the expression that are words -- according to the definition of word characters in the filetype config. Meaning that looking for "+" would not fail only because "match whole word only" is checked, because this setting is irrelevant for this expression. Searching for " = name" could succeed with this option set as expected, but fail on " = names". Am I clear? The current behaviour leads to constantly uncheck this option when irrelevant, which may be dangerous for next search & replace, each time we forget to set it back.
I agree it should be implemented.
-2- record settings The common settings we use (for me: escape sequence, case sensitive, whole word only, not close dialog) are not recorded between sessions and cannot be set in preferences. As a consequence, we need to redefine the dialog for each working session -- and again some missing or error may be dangerous. Would it be possibly to add these settings to the "search" frame in preferences?
We could just save the dialog state, probably no need to add global prefs.
-3- comments & strings A very common task when improving, extending or refactoring code is renaming -- if only for clarity. Too often we'll have to search & replace words that are "key words" for the application, so that they may happen in comments and even in strings. Depending on the case we want the replacement to extend comments and string -- or not at all! Unnoticed and unintended replacements commonly lead to funny but meaningless comments or unpredictable issues because of string changes. A pair of settings to address this point would be great.
Could be useful, as Geany already knows these things for most filetypes. It might be tricky for other filetypes, but even so we could add this option.
About all 3: clean patches welcome.
Regards, Nick