From: Jonas Bernoulli via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 75355@debbugs.gnu.org
Subject: bug#75355: [PATCH 1/1] Improve comment cycling in log-edit
Date: Sat, 04 Jan 2025 23:29:34 +0100 [thread overview]
Message-ID: <8734hy1acx.fsf@bernoul.li> (raw)
In-Reply-To: <86pll2bezh.fsf@gnu.org>
Eli Zaretskii <eliz@gnu.org> writes:
>> Date: Sat, 4 Jan 2025 18:11:08 +0100
>> From: Jonas Bernoulli via "Bug reports for GNU Emacs,
>> the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
>>
>> Save the current message before cycling to older messages, making it
>> possible to cycle back to that initial message.
>
> Thanks, but can you provide some rationale for this? Is the
> assumption that users need to make several commits that all share the
> same comment or something?
That is one use-case for the feature as it exists now, yes. Messages
are already automatically saved once the user either finished or aborts
the commit.
These changes don't really affect that. I consider this additional
automatic saving a bugfix. Without it, a user may start typing a new
message, decide to use a recent message instead, navigate to it but then
change their mind about that, and then they would not be able to go back
to the new message they had already started typing, because it was
discarded when they moved a way from it. By saving the new message when
we move away from it, we make it possible to navigate back to it.
By additionally defining log-edit-save-comment as a command we gain the
ability to save the message at random point. This could, for example,
be useful if we have to use very similar messages in different commits,
potentially across multiple repositories. We could then write the
common part, save it as a "template", edit it some more, and finally
create a first commit. Without the save command we could rely on the
automatic saving at the very end of that process, which would mean that
when creating the second commit we could not use the template as such,
but only template with the modifications for the first commit already
filled in. More editing could be required to go from that to what we
actually want to use in the second commit.
next prev parent reply other threads:[~2025-01-04 22:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-04 16:28 bug#75355: [PATCH 0/1] Improve comment cycling in log-edit Jonas Bernoulli via Bug reports for GNU Emacs, the Swiss army knife of text editors
2025-01-04 17:11 ` bug#75355: [PATCH 1/1] " Jonas Bernoulli via Bug reports for GNU Emacs, the Swiss army knife of text editors
2025-01-04 18:39 ` Eli Zaretskii
2025-01-04 22:29 ` Jonas Bernoulli via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2025-01-05 7:28 ` Eli Zaretskii
2025-01-05 11:37 ` Jonas Bernoulli via Bug reports for GNU Emacs, the Swiss army knife of text editors
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8734hy1acx.fsf@bernoul.li \
--to=bug-gnu-emacs@gnu.org \
--cc=75355@debbugs.gnu.org \
--cc=eliz@gnu.org \
--cc=jonas@bernoul.li \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.