Add shortcut (Ctrl+L) to clear logs

[sunny-Ne5]

Fix #403

[jesseduffield]

Looking good. Two things:

• Could we make this keybinding also available in the side views that render to the main view? I'd prefer to not have to hit enter in order to press ctrl+l
• Looks like the keybinding isn't rendered properly in the keybindings menu (which you get upon pressing '?'

Looks like we need to add a special key to the switch statement in GetKey() in pkg/gui/keybindings.go

[sunny-Ne5]

• Could we make this keybinding also available in the side views that render to the main view? I'd prefer to not have to hit enter in order to press ctrl+l

Should we enable this keybinding for "Containers" or for other side views as well? For this I was planning to add one more entry to GetInitialKeybindings() in keybindings.go. Let me know if there is a better way to do it if it's repetitive.

*Looks like the keybinding isn't rendered properly in the keybindings menu (which you get upon pressing '?'
Is this good?

[jesseduffield]

@sunny-Ne5 Sorry for the late reply! You'll only need the keybinding for containers and services, so feel free to just add an extra two keybindings for that

[peauc]

I want to take a look at this PR to move it forward.
@sunny-Ne5, are you still interested in this PR, you don't mind ?

[sunny-Ne5]

please go ahead

…

On Sun, Jun 16, 2024 at 3:54 AM PÉAU Clément ***@***.***> wrote: I want to take a look at this PR to move it forward. @sunny-Ne5 <https://github.com/sunny-Ne5>, are you still interested in this PR, you don't mind ? — Reply to this email directly, view it on GitHub <#460 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AI7EORF3OVIQZZTORPOWGXLZHS5LPAVCNFSM6AAAAABJMA3TNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZQHE2DMMRTG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[peauc]

I've pulled your changed and have started a branch 👍. Thanks

[peauc]

@jesseduffield requested changes have been made in my pull request 👍

[obfox]

Do you added this feature on different branch?

[peauc]

@obfox hello, sadly the repo is not maintained. All pull requests are merged on my personal fork if you want to get a build with a few fixes.