! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Use Pango native ellipsize; highlight the active window item
Status:
RESOLVED: FIXED
Product:
Xfce4-panel
Component:
Windows Menu

Comments

Description Darren Salt 2006-08-06 15:33:31 CEST
I noticed that the width at which truncation occurs causes display oddness where the "remove workspace" text is somewhat longer than the truncated text. Also, it doesn't properly cope with UTF-8 because it assumes that one character == one byte.

I noticed a question about why the active window isn't highlighted (in #xine) and decided to have a look... this is a trivial enhancement and is most easily done at the same time.
Comment 1 Darren Salt 2006-08-06 15:35:02 CEST
Created attachment 708 
Use Pango native ellipsize; highlight the active window item
Comment 2 Darren Salt 2006-08-06 15:38:28 CEST
See also bug 2127 (xfdesktop's window list menu).
Comment 3 Jasper Huijsmans editbugs 2006-08-07 08:50:03 CEST
Ok, I'll have a look as soon as I can. Sounds like a nice improvement. Thanks.
Comment 4 Jasper Huijsmans editbugs 2006-08-11 19:44:27 CEST
I committed something, a bit simpler than your patch, similar to what the tasklist does. It's in revision 22718. Thanks!
Comment 5 Stefan Stuhr 2006-08-11 20:08:39 CEST
There's a problem with the way the active window is highlighted (bold text): It's the same way that urgent windows are hightlighted.
Comment 6 Jasper Huijsmans editbugs 2006-08-12 07:28:21 CEST
(In reply to comment #5)
> There's a problem with the way the active window is highlighted (bold text):
> It's the same way that urgent windows are hightlighted.
> 

Indeed, there is. Any suggestions for a solution? 
Comment 7 Darren Salt 2006-08-12 12:19:08 CEST
Use underlining?

    if (netk_window_is_active (window))
    { 
        g_string_prepend (label, "<u>");
        g_string_append (label, "</u>");
    }
Comment 8 Jasper Huijsmans editbugs 2006-08-14 08:26:44 CEST
(In reply to comment #7)
> Use underlining?
> 
>     if (netk_window_is_active (window))
>     { 
>         g_string_prepend (label, "<u>");
>         g_string_append (label, "</u>");
>     }
> 

Eww ;-) I really don't like underline.

Instead I made the active window always prelighted, as if selected. Not perfect, but it's the best I can do. It's in SVN revision 22752.

Please reopen if there are still problems with this.

Comment 9 Darren Salt 2006-08-14 19:03:44 CEST
Navigation of the menu via the keyboard breaks the highlighting (but selection of the menu items via the keyboard is broken anyway due to the "activated" signal not being used - see attachment 717  for bug 2104).

Personally, I'd rather have underlining or italic - I don't think that prelight really works there unless you happen to be using a theme which makes the highlighted menu item look somewhat like a highlighted button.
Comment 10 Jasper Huijsmans editbugs 2006-08-14 19:17:04 CEST
Yeah, I saw keyboard breaks the highlighting. I would consider italics, but not underline. 
Comment 11 Jasper Huijsmans editbugs 2006-08-14 19:28:38 CEST
How about revision 22756? It's a fairly subtle effect now, but I think it is ok.
Comment 12 Darren Salt 2006-08-14 19:52:26 CEST
Well, at least it doesn't break anything now :-)

Despite it being perhaps a shade too subtle (unlike, say, underlining) and the fact that tooltips appear for untruncated items, I'm happy to call this fixed...

Bug #2126

Reported by:
Darren Salt
Reported on: 2006-08-06
Last modified on: 2010-11-20

People

Assignee:
Nick Schermer
CC List:
1 user

Version

Version:
Unspecified

Attachments

Additional information