Ticket #1155 (closed bug: fixed)

Opened 17 months ago

Last modified 5 days ago

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 development
Cc: mattlacey, nielx Blocked By:
Platform: All Blocking:

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

bug_1155_fix.diff (280 bytes) - added by mattlacey 4 months ago.
fix

Change History

Changed 16 months ago by wkornewald

  • summary changed from Screen Saver Pref -- Settings aren't detected on-the-fly to Screen Saver Pref -- Settings aren't detected on-the-fly (easy)

Changed 8 months ago by mattlacey

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

Changed 8 months ago by mattlacey

  • cc laceysnr@… added

Forgot to CC myself!

Changed 8 months ago by nielx

  • cc nielx added

It's all yours! Good luck!

Changed 4 months ago by mattlacey

fix

Changed 4 months 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 ;)

Changed 3 months ago by diver

  • cc mattlacey added; laceysnr@… removed
  • component changed from - General to Preferences/ScreenSaver

Could someone take a look at this patch?

Changed 6 days ago by korli

  • owner changed from axeld to korli

Changed 5 days ago by korli

  • status changed from new to closed
  • resolution set to fixed

Applied in r27173. Tested OK.

Note: See TracTickets for help on using tickets.