Bug 8438 – screensaver automatic brightness (re)setting too agressive

Bug 8438 - screensaver automatic brightness (re)setting too agressive

Summary:

screensaver automatic brightness (re)setting too agressive

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Display
Version:	7.0.1
Platform:	PC Other

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-06-15 17:17 UTC by Peter Watkins
Modified:	2008-06-20 16:55 UTC (History)
CC List:	0 users

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 Peter Watkins 2008-06-15 17:17:08 UTC

Prior to the change noted in Bug 7922, a plugin could call showBriefly() to display text on a player, pass a brightness arg, and be sure that the player would display that text for the indicated number of seconds. Now, the brightness requested by showBriefly() is quickly overridden by the player's preferences.

Also this change has broken the very popular AutoDisplay/AutoDim plugin that many of us use to make our players' screens go dark at night.

I believe the patch applied for Bug 7922 is too agressive, and has cause trouble for more customers than it helped. Please consider backing it out and addressing 7922 in a different way.

Comment 1 KDF 2008-06-15 17:26:07 UTC

fix options have already been given for the autodim plugin.  In fact, I use similar modifications (storing and restoring prefs) for local use.

showBriefly override should be fixed.

Comment 2 James Richardson 2008-06-20 09:13:44 UTC

(In reply to comment #1)
> fix options have already been given for the autodim plugin.  In fact, I use
> similar modifications (storing and restoring prefs) for local use.
> 
> showBriefly override should be fixed.
> 

KDF: can you reference the change then close the bug as fixed

Comment 3 KDF 2008-06-20 09:23:35 UTC

I don't have a patch, I'm just pointing out that the original change for bug 7922 is the right one, and further issues with showBriefly and autoDim should be handled by those functions.