On 2/8/07, Andy Parkins <andyparkins@xxxxxxxxx> wrote:
On Thursday 2007, February 08 19:06, Marco Costalba wrote:
> I have applied and pushed all your patches. BTW you are referring to
> qgit4 at Pasky http://repo.or.cz/ site, not the stable qgit in
> kernel.org repo!
My apologies - that was the first one I found when I typed qgit4
repository.
No problem, stable qgit has less layout problems, probably the only
patch I will back port is the fixed font in log viewer.
qgit4 seems to have more layout weirdeness also in patch and file
tabs. I didn't fixed it until because I'm still wondering if it is
enough fixing the layout using stretch factors or bite the bullet and
implement a geometry persistent saving with QSettings (Qt4 is quite
powerful in this regard), of course saved geometry should work both
with maximized and not-maximized window, so perhaps saving
panes/splitters factors is better then save absolute value at least
for 'internal' widget, and use absolute window geometry only for qgit
main window.
> P.S: Yes, I have noted this Qt::convertFromPlainText() misbehaviour,
> but I don't know if are the spaces that are not escaped correctly by
> the function or the QTextEdit rendering engine that eats the
> (escaped) spaces anyway.
I definitely think this is a bug in Qt; QTextBrowser will show the
spaces in HTML mode if <pre> is used, so I don't think it's a fault in
QTextBrowser.
Please add dbg(text) line in git.cpp, just below
Qt::convertFromPlainText(text) call
text.append("\n\n " + colorMatch(c->shortLog(), shortLogRE) +
'\n' + colorMatch(c->longLog(), longLogRE));
}
text = Qt::convertFromPlainText(text);
dbg(text);
// highlight SHA's
//
// added to commit logs, we avoid to call git rev-parse for a
possible abbreviated
// sha if there isn't a leading trailing space or an open parenthesis and,
This will print the converted text as plain string, as you can see
there is 'something' instead of spaces but I cannot guess what.
> P.P.S: Give a try to Bitstream Vera Sans Mono as fixed width font,
> it's muuuuuch better then Monospace IMHO ;-)
No need to persuade me; that's what I already use. Monospace isn't a
font on most systems - it's an alias to the default monospaced font -
and for me it's aliased to Bitstream Mono :-)
It seems to be done by the following in /etc/fonts/
<alias>
<family>Bitstream Vera Sans Mono</family>
<family>DejaVu Sans Mono</family>
<family>Courier</family>
<family>Courier New</family>
<family>Andale Mono</family>
<family>Luxi Mono</family>
<family>Cumberland AMT</family>
<family>Nimbus Mono L</family>
<family>NSimSun</family>
<family>FreeMono</family>
<default><family>monospace</family></default>
</alias>
In case that helps you.
Thanks
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html