Code review comment for lp:~tigrangab/plank/focused_window_font

Revision history for this message
Rico Tzschichholz (ricotz) wrote :

I like it as it looks now. Hiding items for actual visible windows is not a good idea.

Two notes:
* I would only add the focus action for non-active windows.
* explicit use of copy "var font_description = window_item.get_pango_context ().get_font_description ().copy ();"

« Back to merge proposal