Skip to content

horizontal scrolls and other improvements#791

Open
caarlos0 wants to merge 19 commits intomainfrom
horizontal-scrolls
Open

horizontal scrolls and other improvements#791
caarlos0 wants to merge 19 commits intomainfrom
horizontal-scrolls

Conversation

@caarlos0
Copy link
Copy Markdown
Contributor

@caarlos0 caarlos0 commented Jan 7, 2025

pager:

  • search input width
  • horizontal scroll

filter:

  • horizontal scroll

@caarlos0
Copy link
Copy Markdown
Contributor Author

caarlos0 commented Jan 7, 2025

I feel like viewport needs something similar to a sign column as well.

right now it scrolls line numbers and the line selection as well, as they are part of the viewport content...

ccoVeille
ccoVeille reviewed Jan 9, 2025
View reviewed changes
Comment thread filter/options.go Outdated
@caarlos0
wip
b8988fd 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0
wip
6f6a9d2 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
meowgorithm
meowgorithm reviewed Jan 22, 2025
View reviewed changes
Comment thread filter/filter.go Outdated
meowgorithm
meowgorithm reviewed Jan 22, 2025
View reviewed changes
Comment thread pager/pager.go Outdated
@caarlos0 caarlos0 requested a review from a team as a code owner January 23, 2025 13:50
@caarlos0 caarlos0 requested review from csandeep and removed request for a team January 23, 2025 13:50
@caarlos0 caarlos0 mentioned this pull request Jan 23, 2025
Merged
@caarlos0
Copy link
Copy Markdown
Contributor Author

caarlos0 commented Jan 23, 2025

scroll to bottom doesn't work when soft wrapping, fixing

@caarlos0
chore: update
59bb14c 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0
Copy link
Copy Markdown
Contributor Author

caarlos0 commented Jan 23, 2025

ok, this should all be working now.
Check charmbracelet/bubbles#697 for the actual thing :kek:

@caarlos0
chore: fixes
d7ec390
@caarlos0
chore: readme
528a1cf 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0
chore: revert
e1d07d5 
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
This was referenced Jan 28, 2025
Closed
Merged
@caarlos0
Copy link
Copy Markdown
Contributor Author

caarlos0 commented Feb 12, 2025

will leave this for after bubbletea v2

@caarlos0 caarlos0 added the blocked blocked by some other issue and/or pr label Feb 12, 2025
@caarlos0 caarlos0 mentioned this pull request Mar 27, 2025
Open
maraloon added a commit to maraloon/gum that referenced this pull request Aug 23, 2025
@maraloon
fix(choose): set style for selected under cursor (charmbracelet#791)
8779ef6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@meowgorithm meowgorithm meowgorithm left review comments

@csandeep csandeep Awaiting requested review from csandeep csandeep was automatically assigned from charmbracelet/everyone

+2 more reviewers

@ccoVeille ccoVeille ccoVeille left review comments

@aymanbagabas aymanbagabas aymanbagabas approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

blocked blocked by some other issue and/or pr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@caarlos0 @meowgorithm @aymanbagabas @ccoVeille