Bug #7393: Icons are initially smaller than they should in window-picker-applet

Bug #7393

Icons are initially smaller than they should in window-picker-applet

Added by anonym 2014-06-11 06:23:40 . Updated 2015-04-10 14:45:10 .

Status:

Rejected

Priority:

Low

Assignee:

Category:

Camouflage

Target version:

Start date:

2014-06-11

Due date:

% Done:

Feature Branch:

Type of work:

Code

Blueprint:

Starter:

Affected tool:

Deliverable for:

Description

When an icon is displayed in window-picker-applet for the first time (e.g. when starting an application) it’s at ~75% of its size, and expands to the full 100% only after a new icon is displayed in it (e.g. after starting another application).

Subtasks

History

#1 Updated by intrigeri 2014-06-11 07:37:27

I suspect that Alan won’t want to see it as a blocker for 1.1, as patching window-picker-applet is likely needed to fix this bug.

#2 Updated by anonym 2014-06-11 08:37:23

Some more info on this which may be relevant for debugging:

For panel heights >= 30, the initial icon size is always the same, around 30x30 (verified by taking a screen shot and comparing it pixel by pixel).

If I set the panel height to <= 30 pixels, then all icons are opened at full size (also verified by screen shot comparision).

Another way to force a “small” icon to change to its correct full sizeis to change the panel’s height by two steps. One step is not enough. Yet another way is to drag the icon in the list. Not sure if any of this is of much value.

In the sources, src/task-item.c there’s: #define DEFAULT_TASK_ITEM_HEIGHT 26. That’s pretty close to the “30x30” initial size I’ve seen, but still not exact, so perhaps it’s not the cause of the issue.

#3 Updated by anonym 2014-06-11 08:41:43

Assignee changed from alant to anonym

intrigeri wrote:
> I suspect that Alan won’t want to see it as a blocker for 1.1, as patching window-picker-applet is likely needed to fix this bug.

You beat me to it, see https://mailman.boum.org/pipermail/tails-dev/2014-June/006091.html

Perhaps I should have assigned it to myself then, if since alant isn’t very excited at looking at C code.

#4 Updated by anonym 2014-06-11 16:41:41

Category set to Camouflage

#5 Updated by intrigeri 2014-06-12 03:59:43

Status changed from New to Confirmed

(As said on the ML, IMO the first step is to report that bug upstream, unless we’re 100% positive that we’ll be able to have a fix ready very soon… which I somewhat doubt :)