Bug 8042 – Analog VU meter doesn't hide when showbriefly is up

Bug 8042 - Analog VU meter doesn't hide when showbriefly is up

Summary:

Analog VU meter doesn't hide when showbriefly is up

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Player UI
Version:	7.1
Platform:	Macintosh Other

Importance:	-- normal (vote)
Target Milestone:	---
Assigned To:	Unassigned bug - please assign me!

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-05-02 12:24 UTC by Blackketter Dean
Modified:	2008-05-03 07:10 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Blackketter Dean 2008-05-02 12:24:30 UTC

If a new song transitions while the analog vu meter is up (or some other showbriefly occurs) the vu meter and the overlay mash together.  the vu meter should probably be hidden at that time.

Comment 1 Adrian Smith 2008-05-02 13:15:15 UTC

In what cases do you see showbriefly on song transitions with the screensaver active?  

We can probably turn off visualisers during showbriefly but it may impact screen width if the visualiser is a small one on SB2/3.  Would prefer to avoid looking into the visualiser parameters during showbriefy to check for the analogue screensaver only.

Comment 2 Blackketter Dean 2008-05-02 13:22:06 UTC

I can easily reproduce this with the Sleep button, and i've seen it with song info (maybe from a radio station) but I can't see it right now.

For the side and background visualizers, it's not an issue.  The analog VU meter is special because it's intended to replace any other text on the screen.

Comment 3 Blackketter Dean 2008-05-02 13:22:26 UTC

I can easily reproduce this with the Sleep button, and i've seen it with song info (maybe from a radio station) but I can't see it right now.

For the side and background visualizers, it's not an issue.  The analog VU meter is special because it's intended to replace any other text on the screen.

Comment 4 Adrian Smith 2008-05-03 07:10:53 UTC

Fixed in change 19384 - please test.