Details
- Reviewers
kfunk brauch - Group Reviewers
KDevelop - Commits
- R32:2aff26fcaa3d: VCS History dialog: show commit message with clickable urls & fixed font
Diff Detail
- Repository
- R32 KDevelop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Very good to have somebody working on those dialogs, they really need some love. Thanks!
Only question is, do we really need KTextToHtml to just set a fixed font? Can we not simply set a fixed font as the font of the text browser?
Yes, they really feel strange, and possibly need a bigger overhaul in general, to match workflows (mines are not yet met). This is more a drive-by improvement while I look into getting the annotation delegate to show what I want :)
Only question is, do we really need KTextToHtml to just set a fixed font? Can we not simply set a fixed font as the font of the text browser?
Ehm, the KTextToHtml is used to transform all the urls into mark-uped hyperlinks, so QTextBrowser actually displays them as links (see https://api.kde.org/frameworks/kcoreaddons/html/namespaceKTextToHTML.html). Found this method when looking how KDE PIM's messagelib turns plain text into enriched HTML, which calls it.
And while passing in potential HTML text anway, the <tt> felt more normal, as it would spare the code to look-up the system UI font set for fixed-size fonts, incl. ensuring an update of the font if systemsettings change. So with this in mind, would you still prefer some setFont()?
BTW, in an utopian future there will be also text processor plugins which turn "BUG:xyz," "REVIEW:abc" & Co. into clickable links, perhaps even with enriched metadata pulled from the server.