[PATCH] qgit: increase the space between the lanes.

Subsystems: the rest

DORMANTno replies

6 messages, 2 authors, 2016-06-15 · open the first message on its own page

[PATCH] qgit: increase the space between the lanes.

From: Martin Atukunda <hidden>
Date: 2016-06-15 22:42:15

There are probably better ways of doing this.

Signed-off-by: Martin Atukunda <redacted>

---

 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

Re: [PATCH] qgit: increase the space between the lanes.

From: Marco Costalba <hidden>
Date: 2016-06-15 22:42:15

Martin Atukunda wrote:

There are probably better ways of doing this.

Signed-off-by: Martin Atukunda <redacted>

---

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

Re: [PATCH] qgit: increase the space between the lanes.

From: Martin Atukunda <hidden>
Date: 2016-06-15 22:42:15

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 <redacted>

---

 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

Re: [PATCH] qgit: increase the space between the lanes.

From: Martin Atukunda <hidden>
Date: 2016-06-15 22:42:15

On Tuesday 03 January 2006 19:49, Martin Atukunda wrote:

On Tuesday 03 January 2006 18:49, Marco Costalba wrote:

quoted

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 -

Re: [PATCH] qgit: increase the space between the lanes.

From: Marco Costalba <hidden>
Date: 2016-06-15 22:42:15

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

Re: [PATCH] qgit: increase the space between the lanes.

From: Martin Atukunda <hidden>
Date: 2016-06-15 22:42:15

On Wednesday 04 January 2006 01:17, Marco Costalba wrote:

Martin Atukunda wrote:

quoted

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.

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help