Opened 12 years ago

Closed 11 years ago

#1155 closed bug (fixed)

Screen Saver Pref -- Settings aren't detected on-the-fly (easy)

Reported by: mmadia Owned by: korli
Priority: normal Milestone: R1
Component: Preferences/ScreenSaver Version: R1/pre-alpha1
Keywords: Cc: mattlacey, nielx
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

rev. 20674, real hardware.

Screen Saver preflet: When changing Modules, The Module will either need to be [Test]'ed or "Enable Screen Saver" on Fade tab to be cycled. Otherwise Module-based settings will not be read by the pref app / system.

Once the preflet is closed, the settings are committed.

Attachments (1)

bug_1155_fix.diff (280 bytes) - added by mattlacey 11 years ago.
fix

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by wkornewald

Summary: Screen Saver Pref -- Settings aren't detected on-the-flyScreen Saver Pref -- Settings aren't detected on-the-fly (easy)

comment:2 Changed 11 years ago by mattlacey

If nobody has picked this up yet (doesn't appear so!) I think I'll make it my first Haiku contribution.

comment:3 Changed 11 years ago by mattlacey

Cc: laceysnr@… added

Forgot to CC myself!

comment:4 Changed 11 years ago by nielx

Cc: nielx added

It's all yours! Good luck!

Changed 11 years ago by mattlacey

Attachment: bug_1155_fix.diff added

fix

comment:5 Changed 11 years ago by mattlacey

That should sort it - it really didn't take me that long, just lost my build environment and got distracted by travelling and work!

Unemployed now so much more free time ;)

comment:6 Changed 11 years ago by diver

Cc: mattlacey added; laceysnr@… removed
Component: - GeneralPreferences/ScreenSaver

Could someone take a look at this patch?

comment:7 Changed 11 years ago by korli

Owner: changed from axeld to korli

comment:8 Changed 11 years ago by korli

Resolution: fixed
Status: newclosed

Applied in hrev27173. Tested OK.

Note: See TracTickets for help on using tickets.