Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add scrolling to layers pane #478

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

moaimullet
Copy link

Hi,

I made some changes to handle scrolling when the Layers section is active. I learned Go while doing this so I probably did not do everything The Right Way (tm).. You may need to rewrite some of it (not sure if the extra Buffer is needed in LayerSetState for instance). But it seems to work fine.

Regards,
Teague

Related issue: #469

@rroblik
Copy link

rroblik commented Jan 4, 2024

Any news about it ? Latest docker image still have the issue :(

@mark2185
Copy link
Contributor

mark2185 commented Jan 4, 2024

@rroblik build #473 from source, dive hasn't had a new release for a while.

@rroblik
Copy link

rroblik commented Jan 4, 2024

Thanks Luka,

Sorry but I don't find the "how to do" such build, do you have any help link ?

Thanks

@mark2185
Copy link
Contributor

mark2185 commented Jan 5, 2024

@rroblik currently the simplest way to do it would to have golang installed, clone the repo (and checkout that branch), cd into it and run go run main.go <name-of-your-docker-image>.

joschi added a commit to joschi/dive that referenced this pull request Nov 7, 2024
joschi added a commit to joschi/dive that referenced this pull request Nov 7, 2024
@RafaelWO
Copy link

RafaelWO commented Dec 9, 2024

Sorry but I don't find the "how to do" such build, do you have any help link ?

I was able to build dive from source using the Makefile:

make bootstrap
make build

The build artifacts should be under snapshot/

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.

4 participants