Differential D5098

Search tool: no big fonts and less cursive in output view
ClosedPublic
Actions

Authored by kossebau on Mar 18 2017, 2:04 PM.

Details

Reviewers
kfunk
Group Reviewers
KDevelop
Commits
R32:052ca433e7f9: Search tool: no big fonts and less cursive in output view
R33:052ca433e7f9: Search tool: no big fonts and less cursive in output view
Summary

Also switch from "(one match)" to ": 1 match" for consistency
with the Code browser tool view

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Mar 18 2017, 2:04 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptMar 18 2017, 2:04 PM
kossebau added a comment.Mar 18 2017, 2:06 PM

Before:


After:

kfunk accepted this revision.Mar 18 2017, 4:49 PM
kfunk added a subscriber: kfunk.

+1.

Can we git rid off the excessive line spacing, too? Can we enable the uniform row heights property on the view then?

This revision is now accepted and ready to land.Mar 18 2017, 4:49 PM
kossebau added a comment.Mar 18 2017, 5:55 PM
In D5098#95929, @kfunk wrote:

Can we git rid off the excessive line spacing, too? Can we enable the uniform row heights property on the view then?

Not sure yet. I will have some closer look in the next weeks, as I e.g. also would like to fix vertical alignment of linecontent per file of the "Line xyz: linecontent"

Closed by commit R33:052ca433e7f9: Search tool: no big fonts and less cursive in output view (authored by kossebau). · Explain WhyMar 18 2017, 5:58 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 12595

View Options

plugins/grepview/grepoutputmodel.cpp

Loading...