Opened 17 years ago

Closed 16 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:
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 16 years ago.
fix

Download all attachments as: .zip

Change History (9)

comment:1 by wkornewald, 17 years ago

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

comment:2 by mattlacey, 16 years ago

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

comment:3 by mattlacey, 16 years ago

Cc: laceysnr@… added

Forgot to CC myself!

comment:4 by nielx, 16 years ago

Cc: nielx added

It's all yours! Good luck!

by mattlacey, 16 years ago

Attachment: bug_1155_fix.diff added

fix

comment:5 by mattlacey, 16 years ago

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 by diver, 16 years ago

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

Could someone take a look at this patch?

comment:7 by korli, 16 years ago

Owner: changed from axeld to korli

comment:8 by korli, 16 years ago

Resolution: fixed
Status: newclosed

Applied in hrev27173. Tested OK.

Note: See TracTickets for help on using tickets.