#957 - Cannot select a user name in the To field of a private topic in Conversr
| Identifier | #957 |
|---|---|
| Issue type | Minor issue (breaks specific functionality) |
| Title | Cannot select a user name in the To field of a private topic in Conversr |
| Status | Completed |
| Handling member | Chris Graham |
| Addon | General / Uncategorised |
| Description | Hi,
Using an iPad or iPhone in full or mobile mode the drop down doesn't work. In fact it does not work for the Product Version drop down on this form either. Cheers, Ade |
| Steps to reproduce | Go to http://vwgolfmk1.org.uk/ join up if not a member.
Start a new Private topic.
Start typing in the To box.
A small drop down appears but it is never filled with a username.
As long as you know the username you can keep typing and the system users it.
This happens in other areas of the forum as well, I guess it is a common feature. |
| Funded? | No |
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".


Comments
This issue has regressed on our site - I am not sure when it happened though.
Can you detail the changes that are required so that I can reapply it please.
Cheers
Ade
This fixes it, better than it would have been before...
diff --git a/themes/default/templates/JAVASCRIPT.tpl b/themes/default/templates/JAVASCRIPT.tpl
index 92c8d30..7d78871 100644
--- a/themes/default/templates/JAVASCRIPT.tpl
+++ b/themes/default/templates/JAVASCRIPT.tpl
@@ -867,7 +867,7 @@ function browser_matches(code)
switch (code)
{
case 'ios':
- return browser.indexOf('iphone')!=-1;
+ return browser.indexOf('iphone')!=-1 || browser.indexOf('ipad')!=-1;
case 'android':
return browser.indexOf('android')!=-1;
case 'wysiwyg':
diff --git a/themes/default/templates/JAVASCRIPT_AJAX_PEOPLE_LISTS.tpl b/themes/default/templates/JAVASCRIPT_AJAX_PEOPLE_LISTS.tpl
index a719873..bbe315d 100755
--- a/themes/default/templates/JAVASCRIPT_AJAX_PEOPLE_LISTS.tpl
+++ b/themes/default/templates/JAVASCRIPT_AJAX_PEOPLE_LISTS.tpl
@@ -25,6 +25,15 @@ function close_down()
window.currently_doing_list=null;
+add_event_listener_abstract(window,'load',function () {
+ var es=document.getElementsByTagName('input');
+ for (var i=0;i<es.length;i++)
+ {
+ if (es[i].getAttribute('autocomplete')=='off')
+ es[i].setAttribute('autocorrect','off');
+ }
+} );
+
function update_ajax_member_list(target,special,delayed,e)
{
if (typeof e=='undefined') var e=window.event;
@@ -36,6 +45,7 @@ function update_ajax_member_list(target,special,delayed,e)
if (e && enter_pressed(e)) return null;
target.setAttribute('autocomplete','off');
+ target.setAttribute('autocorrect','off');
if (target.disabled) return;
if (!browser_matches('ios'))
The main change is the first, letting iPad detect as iOS.