Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#16538 closed bug (fixed)

Brightness only set to 1st workspace

Reported by: humdinger Owned by: pulkomandy
Priority: normal Milestone: Unscheduled
Component: Drivers/Graphics/intel_extreme Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

This is hrev54590

I used to set the brightness of my screen (using intel_extreme driver) in my UserBootscript. And that worked on all my workspaces. Also, setting the brightness in the Screen prefs worked, setting the brightness of all workspaces.

Now, I suspect after hrev54588, only the brightness of workspace 0 is set, all others are running at a 100% glare. Opening the Screen prefs on such a workspace shows the brightness slider at the reduces value, while the actual screen brightness is 100%.

This is the start of my /boot/home/config/settings/system/app_server/workspaces:

BMessage('asws') {
        columns = int32(0x4 or 4)
        rows = int32(0x1 or 1)
        workspace[0] = BMessage(0x0) {
                screen = BMessage(0x0) {
                        id = int32(0x0 or 0)
                        frame = BRect(l:0.0, t:0.0, r:1919.0, b:1079.0)
                        mode = (type = 'RAWT')(size = 40)
                        brightness = float(0.1569)
                }
                color = int32(0xff986633 or -6789581)
        }
        workspace[1] = BMessage(0x0) {
                screen = BMessage(0x0) {
                        id = int32(0x0 or 0)
                        frame = BRect(l:0.0, t:0.0, r:1919.0, b:1079.0)
                        mode = (type = 'RAWT')(size = 40)
                        brightness = float(1.0000)
                }
                color = int32(0xff986633 or -6789581)
        }
        workspace[2] = BMessage(0x0) {
                screen = BMessage(0x0) {
                        id = int32(0x0 or 0)
                        frame = BRect(l:0.0, t:0.0, r:1919.0, b:1079.0)
                        mode = (type = 'RAWT')(size = 40)
                        brightness = float(1.0000)
                }
                color = int32(0xff986633 or -6789581)
        }

I'm not 100% sure, but I think I remember the "brightness" entry here used to be only under workspace[0], not all of them.

Change History (4)

comment:1 by pulkomandy, 4 years ago

In the settings file the brightness is intentionally stored only in the first workspace, since we want to store a single value and there isn't another convenient place to do that (we still want one per screen, and the list of screens is inside each workspace). It should not be set in other workspaces indeed.

But the brightness setting should be independant of workspaces and video modes now so I don't understand how it can be reset when switching workspaces (yet).

I must have misunderstood something about how app_server manages the workspaces then?

comment:2 by humdinger, 4 years ago

Could change hrev54588 be reverted until somebody figured out what's wrong with it? It's a bit inconvenient having all but workspace[0] at 100%, with no way to tune it down...

comment:3 by pulkomandy, 4 years ago

Resolution: fixed
Status: newclosed

Fixed in hrev54606.

comment:4 by humdinger, 4 years ago

Thank you, thank you, thank you! That's from me and my two eyes... 8)

Note: See TracTickets for help on using tickets.