Opened 6 years ago

Last modified 9 months ago

#1484680 new Feature Patches

Patch for iPhone use

Reported by: chasd Owned by:
Priority: 5 Milestone: later
Component: User Interface Version: git-master
Severity: normal Keywords: iPhone
Cc:

Description

Attached zip file include svn diff from r932 SVN and three new PNG files for an "Open message" button. This is to allow iPhone ( or iPod Touch ) access to RoundCube, where a double-click will not open a message. One line added to the en_US labels file for the button, and one line added to the mail template file of the default skin.

Attachments (3)

iphone_patch.zip (6.1 KB) - added by chasd 6 years ago.
iphone_fix_message_list.diff (613 bytes) - added by damoxc 3 years ago.
mobile_fix_message_list_dblclick.diff (1.2 KB) - added by Kepi 9 months ago.
Patch for support singleclick instead of dblclick on mobile browsers

Download all attachments as: .zip

Change History (14)

Changed 6 years ago by chasd

comment:1 Changed 5 years ago by seansan

  • Milestone set to 0.1.5

Review in 0.1.5

comment:2 Changed 5 years ago by thomasb

  • Milestone changed from 0.1.5 to later

comment:3 Changed 4 years ago by anybody

Is there any progress on this?

I've noticed that in 0.3rc1 the Roundcube interface is still totally unusable on the iPhone due to minor problems...

Very sad that the best mobile browser isn't supported :-(

comment:4 Changed 3 years ago by trisk

The Opera Mini 5 mobile browser also has problems opening messages because double-clicking is not supported. I'm not convinced adding a button which takes up valuable screen space is the best solution, though.

comment:5 Changed 3 years ago by damoxc

I've made a patch that changes the event from dblclick to just a click, since you can't really select a message.

This could be improved (I'll probably do this later on this evening) so it adds a is_mobile_device() so scripts can check and adjust behavior accordingly, and of course include support for more than just the iPhone (Blackberry, Opera Mini etc)

Changed 3 years ago by damoxc

comment:6 Changed 2 years ago by alec

Could someone check this with current version? I see some code for iphone/ipad touch-related.

comment:7 Changed 2 years ago by alec

Just thinking, maybe we could add this button and hide it on non-mobile
browsers using css (that should be possible).

comment:8 Changed 2 years ago by Nikita

With these changes we worked very well

Index: roundcubemail/program/localization/en_US/labels.inc
===================================================================
--- roundcubemail/program/localization/en_US/labels.inc	(revision 932)
+++ roundcubemail/program/localization/en_US/labels.inc	(working copy)
@@ -102,6 +102,7 @@
 // toolbar buttons
 $labels['checkmail']        = 'Check for new messages';
 $labels['writenewmessage']  = 'Create a new message';
+$labels['openmessage']  = 'Open message';
 $labels['replytomessage']   = 'Reply to the message';
 $labels['replytoallmessage'] = 'Reply to sender and all recipients';
 $labels['forwardmessage']   = 'Forward the message';
Index: roundcubemail/skins/default/templates/mail.html
===================================================================
--- roundcubemail/skins/default/templates/mail.html	
+++ roundcubemail/skins/default/templates/mail.html	(working copy)
@@ -112,6 +112,7 @@
<div id="messagetoolbar">
<roundcube:button command="checkmail" type="link" class="button checkmail" classAct="button checkmail" classSel="button checkmailSel" title="checkmail" content=" " />
<roundcube:button command="compose" type="link" class="button compose" classAct="button compose" classSel="button composeSel" title="writenewmessage" content=" " />
+<roundcube:button command="show"  type="link" class="buttonPas show"  classAct="button show" classSel="button showSel" title="openmessage"  content=" " />
<roundcube:button command="reply" type="link" class="buttonPas reply" classAct="button reply" classSel="button replySel" title="replytomessage" content=" " />
<span class="dropbutton">
<roundcube:button command="reply-all" type="link" class="buttonPas replyAll" classAct="button replyAll" classSel="button replyAllSel" title="replytoallmessage" content=" " />

===================================================================
--- roundcubemail/skins/default/mail.css	
+++ roundcubemail/skins/default/mail.css	(working copy)
@@ -142,6 +142,7 @@
#messagetoolbar a.sendSel {
  background-position: -416px -32px;
}

+#messagetoolbar a.show {
+  background-position: -416px 0;
+}
+
+#messagetoolbar a.showSel {
+   background-position: -416px -32px;
+}

#messagetoolbar select.mboxlist
{

greetings

Barbara Vergara
www.fecyl.com

Last edited 2 years ago by Nikita (previous) (diff)

Changed 9 months ago by Kepi

Patch for support singleclick instead of dblclick on mobile browsers

comment:9 Changed 9 months ago by Kepi

Hi, we just run into this issue with recent version (0.8.1).

Mobile browsers and especially tablets become more and more common and our users were unhappy that they can't use it there.

I just modified damoxc's patch with detection of common mobile browser. It simply works out of the box. If you are on mobile/tablet it use singleclick for opening mail, on desktop dblclick still works as before.

It will be great to include this patch in next version.

comment:10 Changed 9 months ago by thomasb

Double-tip to open a message works for me on iPads. Even drag&drop can be done on touch devices. Support for that was added in r3682/subversion. See also #1488558

comment:11 Changed 9 months ago by Kepi

Ok, I can see that my approach was wrong because it disabled single click actions (i.e. flag message).

Unfortunately there is still problem on android tablets - simply doubleclick just resize screen and don't do anything special. Same with drag&drop. I tried to add android detection and modify changes you mentioned in r3682/subversion but didn't work.

Note: See TracTickets for help on using tickets.