Fix the wrong/redundant icon in the prompt bar (#2105)

mine
Jayson Wang 2023-05-28 21:02:17 +08:00 committed by GitHub
parent 91cac5e979
commit d383788859
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 20 deletions

View File

@ -38,7 +38,7 @@ type (
type CmdBuff struct { type CmdBuff struct {
buff []rune buff []rune
suggestion string suggestion string
listeners []BuffWatcher listeners map[BuffWatcher]struct{}
hotKey rune hotKey rune
kind BufferKind kind BufferKind
active bool active bool
@ -52,7 +52,7 @@ func NewCmdBuff(key rune, kind BufferKind) *CmdBuff {
hotKey: key, hotKey: key,
kind: kind, kind: kind,
buff: make([]rune, 0, maxBuff), buff: make([]rune, 0, maxBuff),
listeners: []BuffWatcher{}, listeners: make(map[BuffWatcher]struct{}),
} }
} }
@ -221,7 +221,7 @@ func (c *CmdBuff) Empty() bool {
func (c *CmdBuff) AddListener(w BuffWatcher) { func (c *CmdBuff) AddListener(w BuffWatcher) {
c.mx.Lock() c.mx.Lock()
{ {
c.listeners = append(c.listeners, w) c.listeners[w] = struct{}{}
} }
c.mx.Unlock() c.mx.Unlock()
} }
@ -229,36 +229,24 @@ func (c *CmdBuff) AddListener(w BuffWatcher) {
// RemoveListener removes a listener. // RemoveListener removes a listener.
func (c *CmdBuff) RemoveListener(l BuffWatcher) { func (c *CmdBuff) RemoveListener(l BuffWatcher) {
c.mx.Lock() c.mx.Lock()
defer c.mx.Unlock() delete(c.listeners, l)
c.mx.Unlock()
victim := -1
for i, lis := range c.listeners {
if l == lis {
victim = i
break
}
}
if victim == -1 {
return
}
c.listeners = append(c.listeners[:victim], c.listeners[victim+1:]...)
} }
func (c *CmdBuff) fireBufferCompleted(t, s string) { func (c *CmdBuff) fireBufferCompleted(t, s string) {
for _, l := range c.listeners { for l := range c.listeners {
l.BufferCompleted(t, s) l.BufferCompleted(t, s)
} }
} }
func (c *CmdBuff) fireBufferChanged(t, s string) { func (c *CmdBuff) fireBufferChanged(t, s string) {
for _, l := range c.listeners { for l := range c.listeners {
l.BufferChanged(t, s) l.BufferChanged(t, s)
} }
} }
func (c *CmdBuff) fireActive(b bool) { func (c *CmdBuff) fireActive(b bool) {
for _, l := range c.listeners { for l := range c.listeners {
l.BufferActive(b, c.GetKind()) l.BufferActive(b, c.GetKind())
} }
} }

View File

@ -182,6 +182,7 @@ func (p *Prompt) InCmdMode() bool {
} }
func (p *Prompt) activate() { func (p *Prompt) activate() {
p.Clear()
p.SetCursorIndex(len(p.model.GetText())) p.SetCursorIndex(len(p.model.GetText()))
p.write(p.model.GetText(), p.model.GetSuggestion()) p.write(p.model.GetText(), p.model.GetSuggestion())
p.model.Notify(false) p.model.Notify(false)