Page 1 of 1

Issues redundancy

Posted: Sat Jul 26, 2014 5:24 am
by PB
Seems strange to have "FixMe" and "ToDo" twice in the browser?
See in this screenshot, I think it's better if the red boxed text
should NOT appear, because it already appears in column one.

Since the browser looks for the "FixMe" and "ToDo" keywords,
I suggest those keywords get stripped out before putting the
text in column two.

Image

Re: Issues redundancy

Posted: Sat Jul 26, 2014 8:53 am
by STARGÅTE
You can define an other regex, to hidden TODO, but highlight.

Code: Select all

\bTODO\b(?<display>.*)

Re: Issues redundancy

Posted: Sat Jul 26, 2014 8:55 am
by c4s
I already requested it here: PB 5.30b1 - Issue Browser Coloring and Redundant Text (at first posted as a bug report)

In that thread Demivec apparently also posted a workaround, but currently I'm not able to make it work. Though if I recall correctly it worked fine back then when he first posted it?! There is also a link to another thread which might contain some useful tips regarding this request.

In any way I would also suggest that by default the preinstalled "regular exression" (typo in the preference ;)) are constructed in a way so that the issue text doesn't contain the keyword.


Edit: STARGÅTE posted a solution which seems to work.

Re: Issues redundancy

Posted: Sat Jul 26, 2014 10:52 am
by PB
Thanks c4s, I didn't see the other threads.
But still, it should be officially fixed anyway.

Re: Issues redundancy

Posted: Sat Jul 26, 2014 3:33 pm
by Demivec
PB wrote:Thanks c4s, I didn't see the other threads.
But still, it should be officially fixed anyway.
Here's another thread you didn't see.

Freak describes his additions to address whether or not you want something to show in the description and whether or not it is marked in the source code.

I think that qualifies as having officially fixed it (though it still has at least one bug in its implementation).

Re: Issues redundancy

Posted: Sat Jul 26, 2014 10:23 pm
by PB
> I think that qualifies as having officially fixed it

I mean fixed as in a native setting that we don't have to
go and manually change after a fresh PureBasic install.

Anyway, this removes the text but still leaves the colon in
the text description in the browser... any way to remove
the colon too?

Code: Select all

\bTODO\b(?<display>.*)

Re: Issues redundancy

Posted: Sat Jul 26, 2014 10:55 pm
by c4s
If you always want to type and then remove the colon you can use this:

Code: Select all

 \bTODO:(?<display>.*) 

Re: Issues redundancy

Posted: Sat Jul 26, 2014 10:59 pm
by Demivec
Demivec wrote:Anyway, this removes the text but still leaves the colon in
the text description in the browser... any way to remove
the colon too?
This matches TODO followed by either a space or a colon.

Code: Select all

\bTODO(?>:|\b)(?<display>.*)

Re: Issues redundancy

Posted: Sun Jul 27, 2014 7:08 am
by PB
Thanks c4s and Demivec! :)

Now this just has to be official so we don't need to
specify it with every fresh install of PureBasic. ;)