| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
|---|---|---|---|---|---|---|---|---|---|
| 0002289 | NetSurf | RISC OS-specific | public | 2015-03-15 00:01 | 2015-11-14 21:06 | ||||
| Reporter | Simon Smith | ||||||||
| Assigned To | |||||||||
| Severity | feature | Reproducibility | always | ||||||
| Status | closed | Resolution | won't fix | ||||||
| Platform | OS | RISC OS | OS Version | ||||||
| Product Version | 3.4 | ||||||||
| Target Version | Fixed in Version | ||||||||
| Summary | 0002289: UI inconsistency - don't use no double negatives! | ||||||||
| Description | The RISC OS content config window has checkboxes for 'Hide advertisements', 'Disable pop-up windows' and 'Disable Javascript'. So you have to turn a checkbox /on/ to turn something /off/. That's a double negative, which is confusing UI design. Please reverse the sense of the checkboxes and adjust their names accordingly. Show advertisements (default off, I expect) Allow pop-ups (default off, I guess) Enable Javascript (default on, I presume) There may be other inverted config settings here and there (I also saw 'Disable animations', which should be 'Allow animations'. Always phrase things as a positive action, please, never a double negative. May apply to other platforms. | ||||||||
| Tags | No tags attached. | ||||||||
| Fixed in CI build # | |||||||||
| Reported in CI build # | 2609 | ||||||||
| URL of problem page | RISC OS NetSurf config pages, possibly elsewhere, I haven't looked. | ||||||||
| Attached Files |
| ||||||||
 Relationships |
|
 Relationships |
| Notes |
|
Vincent Sanders (administrator) 2015-03-15 00:22 |
Other frontends )gtk/windows) avoid the double negative issue so this is limited to RISC OS |
Sprow (reporter) 2015-09-07 14:10 |
See attached patch which fixes this on the RISC OS frontend. resources/FatMessages: Update interactive help with Javascript/Animations sense reversed (the other interactive help was already bland enough to avoid double negatives). riscos/con_content.c: Reverse the sense of 'Disable Javascript' to match the internal NetSurf variable sense. riscos/con_image.c: Reverse the sense of 'Disable animation' to match the internal NetSurf variable sense. templates/de,nl,en,fr: Reverse the senses as above, change wording of "Disable <thing>" with a synonym per the GTK 'Control' preferences tab. Add the missing "A0-9." validation to the animation speed writeable (fr/de only). Set the text size to "*" so the Template file compiler fills it in automatically. Move the "Enable animation" checkbox up above the speed setting which it enables so the eye is drawn down the page when the setting is unfaded. |
Dave Higton (developer) 2015-11-14 21:06 |
We discussed this issue at the recent Developer Weekend. There is no perfect answer. The existing scheme has check boxes that are expected NOT to be ticked for the normal case: advertisements allowed, pop-up windows allowed, Javascript enabled. We chose to continue with the check boxes operation in this sense. Sprow, I'm sorry that you have done all that abortive work. |
| Notes |
| Issue History |
|||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-03-15 00:01 | Simon Smith | New Issue | |
| 2015-03-15 00:22 | Vincent Sanders | Note Added: 0000768 | |
| 2015-03-15 00:22 | Vincent Sanders | Severity | minor => feature |
| 2015-03-15 00:22 | Vincent Sanders | Status | new => acknowledged |
| 2015-03-15 00:22 | Vincent Sanders | Category | General => RISC OS-specific |
| 2015-03-15 00:22 | Vincent Sanders | OS | => RISC OS |
| 2015-03-15 00:22 | Vincent Sanders | Product Version | => 3.4 |
| 2015-03-15 00:22 | Vincent Sanders | Description Updated | View Revisions |
| 2015-09-07 14:10 | Sprow | Note Added: 0000939 | |
| 2015-09-07 14:10 | Sprow | File Added: doubleneg.patch | |
| 2015-11-14 21:06 | Dave Higton | Note Added: 0001088 | |
| 2015-11-14 21:06 | Dave Higton | Status | acknowledged => closed |
| 2015-11-14 21:06 | Dave Higton | Resolution | open => won't fix |
| 2015-11-14 21:06 | Dave Higton | Description Updated | View Revisions |
| Issue History |