* bug#72352: Small tab-bar-move-tab-to-group bug fix
@ 2024-07-29 13:20 Ship Mints
2024-07-29 18:16 ` Juri Linkov
0 siblings, 1 reply; 2+ messages in thread
From: Ship Mints @ 2024-07-29 13:20 UTC (permalink / raw)
To: 72352, juri
[-- Attachment #1: Type: text/plain, Size: 1036 bytes --]
This corrects for the condition where a tab, among its set of tabs which
all share the same group (nil or otherwise), is changed to a new group that
doesn't yet exist. The current behavior is to leave that tab where it is
within the "old group" but this is counter to the goal of keeping
group-related tabs "together." The correction I've made is simply to move
that newly grouped tab to the end. I've tested this and been using a
patched form for a while to no ill effect.
(defun tab-bar-move-tab-to-group (&optional tab)
...
* (pos (if beg* (cond
;; Don't move tab when it's already inside group bounds
((and len (>= tab-index beg) (<= tab-index (+ beg len)))
nil)
;; Move tab from the right to the group end
((and len (> tab-index (+ beg len))) (+ beg len 1))
;; Move tab from the left to the group beginning
((< tab-index beg) beg))
* -1))) ; Move to the end*
...
Much obliged,
Stephane
[-- Attachment #2: Type: text/html, Size: 1985 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* bug#72352: Small tab-bar-move-tab-to-group bug fix
2024-07-29 13:20 bug#72352: Small tab-bar-move-tab-to-group bug fix Ship Mints
@ 2024-07-29 18:16 ` Juri Linkov
0 siblings, 0 replies; 2+ messages in thread
From: Juri Linkov @ 2024-07-29 18:16 UTC (permalink / raw)
To: Ship Mints; +Cc: 72352
close 72352 30.0.60
thanks
> This corrects for the condition where a tab, among its set of tabs which
> all share the same group (nil or otherwise), is changed to a new group that
> doesn't yet exist. The current behavior is to leave that tab where it is
> within the "old group" but this is counter to the goal of keeping
> group-related tabs "together." The correction I've made is simply to move
> that newly grouped tab to the end. I've tested this and been using a
> patched form for a while to no ill effect.
Thanks for the fix, now pushed to emacs-30.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-07-29 18:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-29 13:20 bug#72352: Small tab-bar-move-tab-to-group bug fix Ship Mints
2024-07-29 18:16 ` Juri Linkov
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).