commit e31c89e9729d162ca5e015962b45923263d1eee1
from: Mikhail <mp39590@gmail.com>
via: Mark Jamsek <mark@jamsek.dev>
date: Tue Aug 22 05:04:44 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 - beb5455b68cf2c6513db813b7e36635a5685f95e
commit + e31c89e9729d162ca5e015962b45923263d1eee1
blob - c4839d1c9023a068753142f189620d39826b0968
blob + deb04037a78710cda5bd3e24290aa537cce2c2c6
--- tog/tog.c
+++ tog/tog.c
@@ -2498,7 +2498,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,