extrawurst
Display commit description in addition to hash in file view
It changes the following:
- displays the commit name besides the hash in the file view
I followed the checklist:
- [ ] I added unittests
- [x] I ran
make checkwithout errors - [x] I tested the overall application
- [ ] I added an appropriate item to the changelog
im not convinced of this yet, this rarely fits the screen. maybe we just introduce a shortcut to open the current commit detail popup from the files component, this will contain the commit desc. aswell
this rarely fits the screen
There are multiple things that can't be shown fully when the screen is too small to fit them. But most often it fits on a terminal for me.
The main use of this is remembering which commit this is exactly, and by proxy, sometimes which branch. It's easy to forget which commit does the current file view point at, and even the start of the message is usually enough to distinguish between several commits as a mnemonic.
Do you see any substantial downsides to having an at-a-glance disambiguation?
maybe we just introduce a shortcut
Imo there are many shortcuts already, but no consistent model of interaction yet (hence, #1383).
Being able to switch to a commit view would be really nice if it would always work like a toggle, regardless of how the file view was opened, but that would introduce some overlap with the current use of Esc, and some unpredictable (i.e. special-cased) view switching. So off-hand it seems like something that's easier to get wrong than right without some more detailed UX consideration.