git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[[TIG][PATCH v2] 3/3] Revert "Scroll diff with arrow keys in log view"

From
Kumar Appaiah <a.kumar@alumni.iitm.ac.in>
Date
Aug 6, 2013, 04:58 UTC
Message-ID
<1375765101-31461-4-git-send-email-a.kumar@alumni.iitm.ac.in>
In-Reply-To
<1375765101-31461-1-git-send-email-a.kumar@alumni.iitm.ac.in>

This reverts commit 888611dd5d407775245d574a3dc5c01b5963a5ba. This is because, in the re-engineered log view, scrolling the log with the arrows now updates the diff in the diff view when the screen is split. This resembles the earlier behaviour, and is also what users of software like Mutt (which uses the pager view concept) would expect.

Signed-Off-By: Kumar Appaiah <a.kumar@alumni.iitm.ac.in>
Conflicts:
	tig.c
---
 tig.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tig.c b/tig.c
index 256b589..5f564a5 100644
--- a/tig.c
+++ b/tig.c
@@ -1905,7 +1905,6 @@ enum view_flag {
 	VIEW_STDIN		= 1 << 8,
 	VIEW_SEND_CHILD_ENTER	= 1 << 9,
 	VIEW_FILE_FILTER	= 1 << 10,
-	VIEW_NO_PARENT_NAV	= 1 << 11,
 };
 
 #define view_has_flags(view, flag)	((view)->ops->flags & (flag))
@@ -3773,7 +3772,7 @@ view_driver(struct view *view, enum request request)
 
 	case REQ_NEXT:
 	case REQ_PREVIOUS:
-		if (view->parent && !view_has_flags(view->parent, VIEW_NO_PARENT_NAV)) {
+		if (view->parent) {
 			int line;
 
 			view = view->parent;
@@ -4490,7 +4489,7 @@ log_request(struct view *view, enum request request, struct line *line)
 static struct view_ops log_ops = {
 	"line",
 	{ "log" },
-	VIEW_ADD_PAGER_REFS | VIEW_OPEN_DIFF | VIEW_SEND_CHILD_ENTER | VIEW_NO_PARENT_NAV,
+	VIEW_ADD_PAGER_REFS | VIEW_OPEN_DIFF | VIEW_SEND_CHILD_ENTER,
 	sizeof(struct log_state),
 	log_open,
 	pager_read,
-- 
1.8.3.2
Previous: Kumar Appaiah
Message 5 of 5 in “0/3] Refactoring the log view”
  1. Kumar AppaiahAug 6, 2013
  2. [[TIG][PATCH v2] 1/3] Add log_select function to find commit from context in log viewKumar Appaiah, Aug 6, 2013
  3. [[TIG][PATCH v2] 2/3] Display correct diff the context in split log viewKumar Appaiah, Aug 6, 2013
  4. Kumar AppaiahAug 6, 2013
  5. [[TIG][PATCH v2] 3/3] Revert "Scroll diff with arrow keys in log view"Kumar Appaiah, Aug 6, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.