[geany/geany] Find Previous disabled (#1494)

List overview All Threads

newer

older

[geany/geany] HTML comments...

[geany/geany] Geany closing on...

oonet

12 May 2017 12 May '17

12:07 a.m.

Find dialog shows "Next" enabled but "Previous" disabled.

Xubuntu 17.10 with latest version of Geany from repository (1.29)

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494

Attachments:

attachment.htm (text/html — 1.7 KB)

Show replies by date

elextr

12 May 12 May

1:21 a.m.

Doesn't happen for me with latest Geany. Perhaps you need to give more information about what you are doing.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-300986494

oonet

15 May 15 May

2:16 p.m.

Open new document. Select "Find" from "Search" menu (ctrl+f). Next is enabled, Previous is disabled.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-301559097

elextr

5:32 p.m.

Don't have Xubuntu, but doesn't happen here, what versions of GTK and Glib is it using? Can you try the latest version of Geany (1.30.1) from the [PPA](https://launchpad.net/~geany-dev/+archive/ubuntu/ppa).

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-301611219

Matthew Brush

7:57 p.m.

It happens here if you enable regular expression searching, which makes sense since it probably can't match expressions backwards.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-301637015

oonet

11:12 p.m.

Closed #1494.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#event-1083381650

oonet

11:12 p.m.

Good catch! Previous doesn't work with regular expressions enabled, but does work with regular expressions disabled. As you say, makes sense. Fixed!

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-301663588

Richard Taityr

27 Jan 27 Jan

4:08 p.m.

I'm surprised this has remained broken for so long, and further surprised that the issue was closed (at least twice)! Navigating to the previous regex match remains broken in the latest Geany (1.32) and, unfortunately, this appears to be "[by design]" (the same bug was reported in 2011)!

**In any case, Geany is the only text editor I've used which does not support navigating to the previous regex match.**

[by design]: https://sourceforge.net/p/geany/bugs/698/

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-361015382

elextr

5:52 p.m.

@dicktyr its not by design, its because the regular expression library does not support searching backwards so it would require finding all matches forwards from the start of the document and picking the one before the current position and __nobody has written the code__. Pull requests that add the facility are welcome, attempts to shame volunteer contributors are not.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-361022086

Richard Taityr

1 Feb 1 Feb

8:52 p.m.

...

nobody has written the code

hence it seems quite reasonable that this issue should remain open until then and potential writers may appreciate this: https://github.com/google/re2/wiki/WhyRE2

(@elextr by the way, I made no attempt to shame anybody but rather to bring attention to some simple facts)

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-362461278

elextr

8:55 p.m.

@dicktyr the OP closed the issue, not any of the Geany contributors, thats their decision.

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/1494#issuecomment-362461760

2298

Age (days ago)

2564

Last active (days ago)

github-comments@lists.geany.org

10 comments

4 participants

tags (0)

participants (4)

elextr
Matthew Brush
oonet
Richard Taityr