[RFE][PATCH] Display free space on device in panels
Jindrich Novy
jnovy at redhat.com
Fri May 19 12:01:29 UTC 2006
Hi Leonard,
On Thu, 2006-05-18 at 23:42 +0200, Leonard den Ottolander wrote:
> On Thu, 2006-05-18 at 16:36 +0300, Pavel Tsekov wrote:
> > Btw if anyone on this list wants to share his thoughts on this patch - now
> > is the time.
>
> Personally I don't like the highlighting of the free space indicator. I
> find it rather distracting. It suffices that the top of the panel is
> highlighted, no need to highlight that info for the active panel. IMO
> the conditional attrset (REVERSE_COLOR) etc should be removed.
I noticed your reply related to the free space widget highlighting also
on bugzilla. I my opinion it makes the look-and-feel of mc more
consistent when the free space widget is highlighted. The reasons are:
1) Everything what is written directly to the panel frame is highlighted
(current working directory, directory history widget).
2) You are sure that you are looking at the correct free space in the
panel you are working in. When it wouldn't be highlighted, you need
to look up at the panel title to be sure whether it's active since
the the highlighted widgets are on top of the panel. This may save
you one eye movement ;)
It seems that we need a feedback from someone else whether to highlight
or not. If more users are unhappy with the highlighting, I have no
problem with removing it. The current version of the patch is applied in
FC5 if you want to see how it looks like (highlighted version).
Cheers,
Jindrich
--
Jindrich Novy <jnovy at redhat.com>
More information about the mc-devel
mailing list