commit 9403b6951f2a118360a5dc2f49b107ffce74d2e3
from: Mikhail <mp39590@gmail.com>
via: Thomas Adam <thomas@xteddy.org>
date: Wed Aug 23 11:26:30 2023 UTC

tog: don't show the base commit marker in limit view

The limit commit queue is invariably filled with different commits to the
queue containing all commits, so the index of the work tree's base commit in
the real queue corresponds to a different commit in the limit queue. As such,
the marker is drawn on an incorrect commit. Rather than fix this to draw the
marker on the correct commit if it happens to be in the limit queue, don't
draw the marker at all in limit view. As pointed out by Mikhail on the list,
this information is not wanted in the limit view. Patch and report by Mikhail.

commit - d95d69074c849c2f95e9f12f2cd4403a60ba30fe
commit + 9403b6951f2a118360a5dc2f49b107ffce74d2e3
blob - 8ff70054ebddda064423a1c2a3f8908625d6bc0f
blob + 063b2f8528680c8325f7441f457de59541099daf
--- tog/tog.c
+++ tog/tog.c
@@ -2502,7 +2502,7 @@ draw_commit(struct tog_view *view, struct commit_queue
 	while (col < avail && author_width < author_display_cols + 2) {
 		if (tog_base_commit.marker != GOT_WORKTREE_STATE_UNKNOWN &&
 		    author_width == marker_column &&
-		    entry->idx == tog_base_commit.idx) {
+		    entry->idx == tog_base_commit.idx && !s->limit_view) {
 			tc = get_color(&s->colors, TOG_COLOR_COMMIT);
 			if (tc)
 				wattr_on(view->window,