Review Request: Fix bug 308846 Clicking on messages does not move the cursor to that location.

Morten Volden mvolden2 at gmail.com
Wed Dec 19 19:48:16 UTC 2012


onsdag den 19. december 2012 skrev Andreas Pakulat :

> Hi,
>
> On Wed, Dec 19, 2012 at 3:28 PM, Jeremy Murphy
> <jeremy.william.murphy at gmail.com <javascript:;>> wrote:
> > On 15 December 2012 21:06, Andreas Pakulat <apaku at gmx.de <javascript:;>>
> wrote:
> >> > I find having both the notice and warning lines of text coming up
> orange
> >> > to
> >> > be confusing and distracting.  Are those colours configurable somehow?
> >> > Would be difficult to make them so?
> >>
> >> Its indeed bogus that warnings and information use the same color
> >> role, they should definitly be different. I've changed it now to use
> >> the Link text color from the color scheme. I think that matches better
> >> than using Active text since its not active itself or new or
> >> something, just something thats clickable to go 'elsewhere' from
> >> clicking.
> >
> > The link identity is definitely an improvement, thanks.  However, I think
> > you have accidentally introduced a bug in doing so, as I have seen an
> error
> > message appear in the link colour, although *most* of them appear
> correctly.
> > I'll try to narrow it down.  Cheers.
>
> I bet this was the same before, if you look at the commit at
>
> http://quickgit.kde.org/?p=kdevplatform.git&a=commitdiff&h=8b6bfba875a1d0a27aeadeb7fcafbdc5583e6eed
> you'll notice that I did not touch the matching of lines at all. If I
> understand the filters correctly some 'parts' of an error may actually
> be highlighted as info-lines because they do not contain the 'error'
> keyword but refer to locations nonetheless. I guess errors from
> templates and errors which are coming from included files fall into
> this category.
>
> Andreas
> _______________________________________________
> KDevelop-devel mailing list
> KDevelop-devel at kde.org <javascript:;>
> https://mail.kde.org/mailman/listinfo/kdevelop-devel
>


-- 
- When the split is pulled, mr. Grenade is no longer our friend
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121219/e1dd235e/attachment.html>


More information about the KDevelop-devel mailing list