Context Navigation

#254 closed bug (fixed)

Wrongly enabled buttons in apps/prefs (easy)

Reported by:	diver	Owned by:	aldeck
Priority:	low	Milestone:	R1
Component:	Applications	Version:
Keywords:		Cc:
Blocked By:		Blocking:
Platform:	All

Description (last modified by wkornewald)

There are several apps/prefs which enable some buttons on the first start, when i didn't change anything. Here is this list:

Fonts - Default
Keyboard - Default
Keymap - Revert, Use
Mouse - Default
Scroll Bar - Default
Virtual memory - Default

Also, when you start Sounds, you could see how butons becomes disabled Sounds - Play, Stop.

Attachments (5)

Anxiety - Ticket 254 - 20070816.diff (16.4 KB ) - added by anxiety 17 years ago.: Diff file
20070816 - Ticket 254b.diff (1.1 KB ) - added by anxiety 17 years ago.: Diff file, sounds disables correctly in this one
20070819 - Ticket 254c.diff (16.0 KB ) - added by anxiety 17 years ago.: Diff file, contains changes for Appearance, Menu, and Tracker
menu.diff (16.0 KB ) - added by aldeck 17 years ago.: Menu preflet should behave correctly with this.
menu2.diff (606 bytes ) - added by aldeck 17 years ago.: Corrects a little mistake in menu/ColorWindow.cpp

Download all attachments as: .zip

Change History (33)

comment:1 by axeld, 19 years ago

Owner:	changed from sikosis to axeld

comment:2 by korli, 19 years ago

bug_group:	→ developers

comment:3 by axeld, 18 years ago

Description:	modified (diff)
Platform:	→ All
Priority:	normal → low

comment:4 by axeld, 18 years ago

Description:	modified (diff)

comment:5 by wkornewald, 18 years ago

Description:	modified (diff)

Screen should work correctly after my changes.

comment:6 by wkornewald, 18 years ago

Summary:	Wrongly enabled buttons in apps/prefs → Wrongly enabled buttons in apps/prefs (easy)

by anxiety, 17 years ago

Attachment:	Anxiety - Ticket 254 - 20070816.diff added

Diff file

comment:7 by anxiety, 17 years ago

I've attached a diff file that fixes the behavior of the default buttons, and hopefully handles the sounds issue satisfactory.

I did not make any changes to Keymap as the revert button doesn't do anything at all (with a note saying the original did not do anything either.) Perhaps that would be a good candidate for its own ticket?

by anxiety, 17 years ago

Attachment:	20070816 - Ticket 254b.diff added

Diff file, sounds disables correctly in this one

comment:8 by axeld, 17 years ago

Indeed, I think Keymap deserves its own ticket. Thanks!

comment:9 by axeld, 17 years ago

Resolution:	→ fixed
Status:	new → closed

And since ScrollBar has been removed, I'm closing this ticket now. Fixed in hrev21992.

comment:10 by diver, 17 years ago

Resolution:	fixed
Status:	closed → reopened

Oh yeah, E-mail pref app still need to be fixed, it was not in the image a year ago, so it didn't made up in this list in the first place.

comment:11 by diver, 17 years ago

Also add to this list: Menu Screen Tracker Preferences

comment:12 by diver, 17 years ago

Oops, i forgot about Appearance... So it must be

Appearance E-mail Menu Screen Tracker Preferences

by anxiety, 17 years ago

Attachment:	20070819 - Ticket 254c.diff added

Diff file, contains changes for Appearance, Menu, and Tracker

comment:13 by anxiety, 17 years ago

I was able to get Appearance, Menu and Tracker working. Screen seems to work fine as long as the screen isn't in VESA mode. When in VESA there seems to be something going on with the refresh rate, but I wasn't able to figure out what. Email is going to take me some time as its more complex.

After looking at tracker, I think the majority of pref apps could benefit from using a consistent settings setup like tracker has.

comment:14 by axeld, 17 years ago

Thanks! In case you didn't notice, your patch was applied in hrev22049.

comment:15 by diver, 17 years ago

I found some problems: Menu: Selecting "Ctrl as Shortcut key" and back doesn't disable Revert button.

Tracker preferences: Selecting "Show Volumes on Desktop" (which is default) enables Default button. This could be seen only once, to reproduce you have to reboot.

And Background have Revert and Apply enabled.

by aldeck, 17 years ago

Attachment:	menu.diff added

Menu preflet should behave correctly with this.

comment:16 by aldeck, 17 years ago

I hope that my patch is ok, i could have gone further in encapsulating all menu_info calls in the MenuSettings class and factoring/cleaning here and there, but i didn't want to make too much changes in one shot.

Going on to check the Screen preflet revert button, which should be disabled on start.

follow-up: 18 comment:17 by stippi, 17 years ago

Works great, thanks a lot! Menu patch applied in hrev23296.

in reply to: 17 comment:18 by aldeck, 17 years ago

Replying to stippi:

Works great, thanks a lot! Menu patch applied in hrev23296.

You're welcome! Thanks for the quick code review :)

by aldeck, 17 years ago

Attachment:	menu2.diff added

Corrects a little mistake in menu/ColorWindow.cpp

comment:19 by aldeck, 17 years ago

menu2.diff applied in hrev23708

follow-up: 21 comment:20 by diver, 17 years ago

There is still a problem with Tracker preferences.

in reply to: 20 comment:21 by aldeck, 17 years ago

Replying to diver:

There is still a problem with Tracker preferences.

Didn't have much time these days. I just found the problem for Tracker settings. Will commit soon :-) Screen and Keymap still need some work though.

comment:22 by aldeck, 17 years ago

Tracker settings Revert/Default buttons should now behave corectly with hrev24008. Note that Mount Settings can't be defaulted/reverted though, since it's handled quite differently.