This reverts commit d38706da.

Changed back, as changed loop didn't do same as original did

Also assume p_item->i_es is set correctly

- use switch/case instead of if/else if/else if
- factor out HandleEditBoxKey() and HandleCommonKey()
- return bool instead of int

Use one line per switch label when it improves readability
Also explicit the sign of position increment even when positive so +x and -x are aligned

And cleanup includes of StandardPanel

Remove friendship
Factorisation of CoverArt
Reduction of class footprints
virtual and protected correctness
Forward declaration when possible

It's useless and create bogus empty commands, that output ugly blank
lines and crap during login

--no-color / --color is a fine enough control

ManageSlider() -> ChangePosition(), now takes the position increment as (signed) argument
Don't show percentage of media in Position (elapsed/total already show this information)

factor out file/directory detection code
remove the need for {OPEN,SEARCH}_CHAIN_SIZE defines

remove code to workaround lack of unicode support

Also statically allocate open and search buffers

align 'case' on the same level than 'switch'
use an inline function to switch boxes