Skip to content

#21 render comments#34

Open
Bakhomious wants to merge 6 commits into
mainfrom
21-render-comments
Open

#21 render comments#34
Bakhomious wants to merge 6 commits into
mainfrom
21-render-comments

Conversation

@Bakhomious
Copy link
Copy Markdown
Contributor

No description provided.

@Bakhomious Bakhomious force-pushed the 21-render-comments branch 5 times, most recently from 0782e3f to fd4f987 Compare May 17, 2026 17:14
@Bakhomious Bakhomious force-pushed the 21-render-comments branch 3 times, most recently from 0fdb500 to 9483654 Compare May 17, 2026 19:48
@Bakhomious Bakhomious force-pushed the 21-render-comments branch from 9483654 to a3ffaa4 Compare May 18, 2026 05:39
Comment thread e2e/mock_server.go
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps it's time to refactor this

Comment thread internal/jira/adf_comment.go
@Bakhomious Bakhomious force-pushed the 21-render-comments branch 2 times, most recently from 2f0db3f to 167f7bf Compare May 18, 2026 17:34
Comment on lines +129 to +150
func (m *CommentInputModel) handleKey(msg tea.KeyPressMsg) (*CommentInputModel, tea.Cmd) {
// Handle suggestion navigation
if m.showSuggest && len(m.suggestions) > 0 {
switch {
case key.Matches(msg, keyDown):
m.selectedIdx = (m.selectedIdx + 1) % len(m.suggestions)
return m, nil
case key.Matches(msg, keyUp):
m.selectedIdx--
if m.selectedIdx < 0 {
m.selectedIdx = len(m.suggestions) - 1
}
return m, nil
case key.Matches(msg, keyTab):
m.acceptSuggestion()
return m, nil
case key.Matches(msg, keyEsc):
m.showSuggest = false
m.suggestions = nil
return m, nil
}
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

List component?

@Bakhomious Bakhomious force-pushed the 21-render-comments branch from 167f7bf to 0a6c486 Compare May 19, 2026 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant