Awn

Generic icon does not scale to bar size

Bug #159291 reported by Michael Rooney
18
Affects Status Importance Assigned to Milestone
Awn
Fix Released
Low
black
Awn 0.2.4-milestone

Bug Description

The generic icon for when a window does not have its own does not resize properly and stays at the 48px height. This affects 0.2-stable-testing r157 but isn't specific to it, it has occurred for as long as I can remember. See attached screen shot.

Revision history for this message
Michael Rooney (mrooney) wrote :
Revision history for this message
Julien Lavergne (gilir) wrote :

Thanks for the report. Confirmed.

Changed in awn:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Michael Rooney (mrooney) wrote :

I would argue this should be Medium importance but since you specifically just made it Low I won't change it myself. But the generic AWN icon not understanding the bar size of AWN seems like a pretty basic thing that would be expected to work and could make users find AWN unpolished.

Revision history for this message
black (blackborn) wrote :

I wrote a solution for it into my libawn-draw-effects branch

(So on the next merge with the trunk, it will be solved)

Changed in awn:
assignee: nobody → blackborn
status: Confirmed → In Progress
Revision history for this message
Mark Lee (malept) wrote :

black: Could you post a patch here? The diff using launchpad's codebrowse is rather large, given that you merged and you added that solution at the same time.

Revision history for this message
black (blackborn) wrote :

You have to change only one line, see patch

Revision history for this message
Michal Hruby (mhr3) wrote :

Thanks for patch.

Changed in awn:
status: In Progress → Fix Committed
Revision history for this message
Vadim Peretokin (vperetokin) wrote :

When can this be taken into effect? (if it is already, my bad. I'm using reaocards repository for awn and still have htis issue).

Mark Lee (malept)
Changed in awn:
milestone: none → 0.2.4
Revision history for this message
Michael Rooney (mrooney) wrote :

Yes it has been fixed for quite awhile. I would have thought it was in 0.2.1.

Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :

New release: https://edge.launchpad.net/awn/0.2/0.2.4/

Changed in awn:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Patches

Remote bug watches

Bug watches keep track of this bug in other bug trackers.