* [PATCH] qgit: increase the space between the lanes. @ 2006-01-03 11:15 Martin Atukunda 2006-01-03 15:49 ` Marco Costalba 0 siblings, 1 reply; 6+ messages in thread From: Martin Atukunda @ 2006-01-03 11:15 UTC (permalink / raw) To: git There are probably better ways of doing this. Signed-off-by: Martin Atukunda <matlads@dsmagic.com> --- src/mainimpl.cpp | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) 28ddfe1297a5e1e212fc35c3419cf8364415847e diff --git a/src/mainimpl.cpp b/src/mainimpl.cpp index d7c4e39..1b1f0f0 100644 --- a/src/mainimpl.cpp +++ b/src/mainimpl.cpp @@ -1409,7 +1409,7 @@ void MainImpl::setupPixmaps() { // little hack to read items height new QListViewItem(listViewLog); ph = listViewLog->firstChild()->height(); - pw = ph / 2; + pw = ph / 2 + 4; listViewLog->clear(); // remove and deletes items QPixmap cm(pw, ph); cm.fill(); @@ -1443,17 +1443,17 @@ void MainImpl::setupPixmaps() { case MERGE_FORK: p.drawLine(P_90, P_270); p.drawLine(P_180, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2, ph/2); break; case MERGE_FORK_R: p.drawLine(P_90, P_270); p.drawLine(P_180, P_OR); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2, ph/2); break; case MERGE_FORK_L: p.drawLine(P_90, P_270); p.drawLine(P_OR, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2, ph/2); break; case JOIN: p.drawLine(P_90, P_270); -- 1.0.GIT ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] qgit: increase the space between the lanes. 2006-01-03 11:15 [PATCH] qgit: increase the space between the lanes Martin Atukunda @ 2006-01-03 15:49 ` Marco Costalba 2006-01-03 16:49 ` Martin Atukunda 0 siblings, 1 reply; 6+ messages in thread From: Marco Costalba @ 2006-01-03 15:49 UTC (permalink / raw) To: Martin Atukunda; +Cc: git Martin Atukunda wrote: > There are probably better ways of doing this. > > Signed-off-by: Martin Atukunda <matlads@dsmagic.com> > > --- > Thanks for the patch. Almost applied ;-) The incresed lines spacing is good. But unluckly the merge/fork symbols don't appear as good, at least in my box: http://digilander.libero.it/mcostalba/qgitOriginal.png http://digilander.libero.it/mcostalba/qgitModified.png If it _could_ be possible to fill with color the merge symbols it would be great. Do you mind to do it yourself? if not I will do that. Thanks Marco ___________________________________ Yahoo! Mail: gratis 1GB per i messaggi e allegati da 10MB http://mail.yahoo.it ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qgit: increase the space between the lanes. 2006-01-03 15:49 ` Marco Costalba @ 2006-01-03 16:49 ` Martin Atukunda 2006-01-03 17:07 ` Martin Atukunda 0 siblings, 1 reply; 6+ messages in thread From: Martin Atukunda @ 2006-01-03 16:49 UTC (permalink / raw) To: git On Tuesday 03 January 2006 18:49, Marco Costalba wrote: > > If it _could_ be possible to fill with color the merge symbols it would be > great. > > Do you mind to do it yourself? if not I will do that. Something like this? --- [PATCH] qgit: increase the space between the lanes. This commit increases the space between the lanes. It also cators/adjusts for the look of the merge/fork symbols. Signed-off-by: Martin Atukunda <matlads@dsmagic.com> --- src/mainimpl.cpp | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) eb82c3b5bda7970bf743a3b1fa9df8a943f3b9ed diff --git a/src/mainimpl.cpp b/src/mainimpl.cpp index d7c4e39..9430e1b 100644 --- a/src/mainimpl.cpp +++ b/src/mainimpl.cpp @@ -1409,7 +1409,7 @@ void MainImpl::setupPixmaps() { // little hack to read items height new QListViewItem(listViewLog); ph = listViewLog->firstChild()->height(); - pw = ph / 2; + pw = ph / 2 + 4; listViewLog->clear(); // remove and deletes items QPixmap cm(pw, ph); cm.fill(); @@ -1443,17 +1443,17 @@ void MainImpl::setupPixmaps() { case MERGE_FORK: p.drawLine(P_90, P_270); p.drawLine(P_180, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2-1, ph/2); break; case MERGE_FORK_R: p.drawLine(P_90, P_270); p.drawLine(P_180, P_OR); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2-1, ph/2); break; case MERGE_FORK_L: p.drawLine(P_90, P_270); p.drawLine(P_OR, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.drawRect(pw/4+1, ph/4, pw/2-1, ph/2); break; case JOIN: p.drawLine(P_90, P_270); -- 1.0.6-g58e3 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] qgit: increase the space between the lanes. 2006-01-03 16:49 ` Martin Atukunda @ 2006-01-03 17:07 ` Martin Atukunda 2006-01-03 22:17 ` Marco Costalba 0 siblings, 1 reply; 6+ messages in thread From: Martin Atukunda @ 2006-01-03 17:07 UTC (permalink / raw) To: git On Tuesday 03 January 2006 19:49, Martin Atukunda wrote: > On Tuesday 03 January 2006 18:49, Marco Costalba wrote: > > If it _could_ be possible to fill with color the merge symbols it would > > be great. > > > > Do you mind to do it yourself? if not I will do that. > > Something like this? Sorry, but it appears that the patches i've been sending out work only for certain font sizes! Hmmm. see these 3 screenshots (with the previous patch): http://mail.ds.co.ug/~matlads/qgit/ Hmm, I need to re-think how to do this properly. Any ideas? - Martin - ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] qgit: increase the space between the lanes. 2006-01-03 17:07 ` Martin Atukunda @ 2006-01-03 22:17 ` Marco Costalba 2006-01-04 8:24 ` Martin Atukunda 0 siblings, 1 reply; 6+ messages in thread From: Marco Costalba @ 2006-01-03 22:17 UTC (permalink / raw) To: Martin Atukunda; +Cc: git Martin Atukunda wrote: > > Hmm, I need to re-think how to do this properly. Any ideas? > Hi Martin, What about this? If it is Ok for you I will push the change. Marco diff --git a/src/mainimpl.cpp b/src/mainimpl.cpp index 2f26b01..40a22e8 100644 --- a/src/mainimpl.cpp +++ b/src/mainimpl.cpp @@ -1451,7 +1451,7 @@ void MainImpl::setupPixmaps() { // little hack to read items height new QListViewItem(listViewLog); ph = listViewLog->firstChild()->height(); - pw = ph / 2; + pw = 3 * ph / 4; listViewLog->clear(); // remove and deletes items QPixmap cm(pw, ph); cm.fill(); @@ -1470,12 +1470,13 @@ void MainImpl::setupPixmaps() { p.begin(pm); p.setPen(QPen(colors[i % COLORS_NUM], 2)); int type = (i / COLORS_NUM) + 1; + QBrush myBrush(QBrush(colors[i % COLORS_NUM], Qt::SolidPattern)); switch(type) { case ACTIVE: p.drawLine(P_90, P_270); p.save(); p.setPen(QPen(colors[i % COLORS_NUM], 1)); - p.setBrush(QBrush(colors[i % COLORS_NUM], Qt::SolidPattern)); + p.setBrush(myBrush); p.drawEllipse(pw/8, ph/8, 6*pw/8, 3*ph/8); p.restore(); break; @@ -1485,17 +1486,20 @@ void MainImpl::setupPixmaps() { case MERGE_FORK: p.drawLine(P_90, P_270); p.drawLine(P_180, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.fillRect (pw/4, ph/4, pw/2, ph/2, myBrush); + p.drawRoundRect(pw/4, ph/4, pw/2, ph/2, 25, 25); break; case MERGE_FORK_R: p.drawLine(P_90, P_270); p.drawLine(P_180, P_OR); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.fillRect (pw/4, ph/4, pw/2, ph/2, myBrush); + p.drawRoundRect(pw/4, ph/4, pw/2, ph/2, 25, 25); break; case MERGE_FORK_L: p.drawLine(P_90, P_270); p.drawLine(P_OR, P_0); - p.drawRect(pw/4, ph/4, pw/2, ph/2); + p.fillRect (pw/4, ph/4, pw/2, ph/2, myBrush); + p.drawRoundRect(pw/4, ph/4, pw/2, ph/2, 25, 25); break; case JOIN: p.drawLine(P_90, P_270); ___________________________________ Yahoo! Mail: gratis 1GB per i messaggi e allegati da 10MB http://mail.yahoo.it ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] qgit: increase the space between the lanes. 2006-01-03 22:17 ` Marco Costalba @ 2006-01-04 8:24 ` Martin Atukunda 0 siblings, 0 replies; 6+ messages in thread From: Martin Atukunda @ 2006-01-04 8:24 UTC (permalink / raw) To: git On Wednesday 04 January 2006 01:17, Marco Costalba wrote: > Martin Atukunda wrote: > > Hmm, I need to re-think how to do this properly. Any ideas? > > Hi Martin, > > > What about this? > > If it is Ok for you I will push the change. > Looks Good. Please apply. - Martin - -- Due to a shortage of devoted followers, the production of great leaders has been discontinued. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2006-01-04 8:25 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2006-01-03 11:15 [PATCH] qgit: increase the space between the lanes Martin Atukunda 2006-01-03 15:49 ` Marco Costalba 2006-01-03 16:49 ` Martin Atukunda 2006-01-03 17:07 ` Martin Atukunda 2006-01-03 22:17 ` Marco Costalba 2006-01-04 8:24 ` Martin Atukunda
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).
