Tested changing settings
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Branch
- kaccess-kconfigxt
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 19983 Build 20001: arc lint + arc unit
Comment Actions
I read kaccess.cpp and commented on where the defaults diverge, there could be more. There's also some keys that have typos and/or don't match.
This shows quite impressively why having this stuff in a central place is useful :)
Given kaccess.cpp is what does the handling, ultimately its defaults will be what is being used, not the ones in the KCM.
| kcms/access/kaccess_settings.kcfg | ||
|---|---|---|
| 13 ↗ | (On Diff #71719) | lol Can you give this a better name, like "customBell" (Arts was the KDE 3 sound server) |
| 17 ↗ | (On Diff #71719) | Likewise, customBellFile |
| 46 ↗ | (On Diff #71719) | "stickied" is not a word but I can see that "stuck" implies something else |
| 55 ↗ | (On Diff #71719) | The default is false |
| 61 ↗ | (On Diff #71719) | "key" or whatever that k stands for? |
| 76 ↗ | (On Diff #71719) | The default is false |
| 80 ↗ | (On Diff #71719) | The default is false |
| 84 ↗ | (On Diff #71719) | The default is false |
| 97 ↗ | (On Diff #71719) | BounceKeysRejectBeep |
| 121 ↗ | (On Diff #71719) | There is only kNotifyAccessX (note the X) |
| 142 ↗ | (On Diff #71719) | This is actually being calculated referencing "interal" in kaccess code: mouseGroup.readEntry("MKTimeToMax", (5000 + interval / 2) / interval); |
| 144 ↗ | (On Diff #71719) | The key is MKTimeToMax |
| 148 ↗ | (On Diff #71719) | Defaults to interval |
| 155 ↗ | (On Diff #71719) | profil*e* |
Comment Actions
Since the other diff seems quite advanced already I am abandonning this one.
| kcms/access/kaccess_settings.kcfg | ||
|---|---|---|
| 55 ↗ | (On Diff #71719) | Well no : see defaults() "ui.stickyKeysBeep->setChecked(true);" or load() "keyboardGroup.readEntry("StickyKeysBeep", true)" |
| 76 ↗ | (On Diff #71719) | No see default() " ui.slowKeysPressBeep->setChecked(true);" or load() |
| 80 ↗ | (On Diff #71719) | No ui.slowKeysAcceptBeep->setChecked(true); |
| 84 ↗ | (On Diff #71719) | Same. |
| 97 ↗ | (On Diff #71719) | What do you mean ? |
| 121 ↗ | (On Diff #71719) | What do you mean ? |
| 142 ↗ | (On Diff #71719) | Yes and it the default value taking into account the default settings : int((5000 + 5/2) / 5) = 1000 |
| 144 ↗ | (On Diff #71719) | No, there are two similar keys MK-TimeToMax and MKTimeToMax, hence your confusion. |