search

blackberry / webworks-gui

GUI Tool for WebWorks 2.0
https://developer.blackberry.com/html5/
Apache License 2.0
3 stars 5 forks source link

Access List section - "Allow subdomains" checkbox does not work #202

Open jamesjhedley opened 10 years ago

jamesjhedley commented 10 years ago

-open config page -add whitelist for some random URL (i.e. http://www.google.ca) -click "add access"

access list entry added successfully

-now check the "allow subdomains" check box -save

Expected -changes are saved and not lost

Actual -changes are lost

rcruz commented 10 years ago

The checkbox on existing entries should not be editable.

jamesjhedley commented 10 years ago

Might actually be useful to let it remain editable, otherwise the user has to re-enter the entire URL just to allow sub domains.

rcruz commented 10 years ago

The user has to re-enter the URl to change the URL. We don't currently have support for editing access entries, but it can always be implemented later.

rcruz commented 10 years ago

Checkboxes cannot be set to readonly, but they can be disabled. Being disabled fades the checkbox, which makes it hard to see on some browsers. An image of a checked box can be used instead. Recommending this change to be in a future release.

jamesjhedley commented 10 years ago

Agreed. We shouldn't be using a checkbox if you can't actually set the value. An image or some text that shows allow subdomains has been set would be best.