Last modified: 2014-07-10 20:19:50 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68999, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66999 - VisualEditor Mobile: References/images require double tap to be selected on iOS Safari
VisualEditor Mobile: References/images require double tap to be selected on i...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
Mobile (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2014-07-03
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-23 21:28 UTC by Juliusz Gonera
Modified: 2014-07-10 20:19 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Double tap required (3.13 MB, video/quicktime)
2014-06-23 21:28 UTC, Juliusz Gonera
Details

Description Juliusz Gonera 2014-06-23 21:28:08 UTC
Created attachment 15702 [details]
Double tap required

If you open VE on mobile and want to edit an existing reference, you need to tap it twice. This is really hard, especially given that the tap target is quite small (which is a separate issue).

Tested and recorder in iOS Safari 7.1.1. Does not happen on Chrome for Android (just a single tap needed there).
Comment 1 Juliusz Gonera 2014-06-23 21:57:53 UTC
Ignore wrong positioning of mobile context (second toolbar), it's fixed now. To actually see the context though you need those patches:
https://gerrit.wikimedia.org/r/#/c/141085/
https://gerrit.wikimedia.org/r/#/c/141087/
Comment 2 James Forrester 2014-06-24 01:24:51 UTC
Pushing to Ed because it appears to be a shield/highlight issue.
Comment 3 Ed Sanders 2014-06-24 15:36:12 UTC
Isn't this fixed then (comment 1)?
Comment 4 James Forrester 2014-06-25 00:27:02 UTC
(In reply to Ed Sanders from comment #3)
> Isn't this fixed then (comment 1)?

… no?

Comment 1 says that the context being useless ("the tap target is quite small") is  fixed by some other patches and you should ignore.

Comment 0 is mostly about "If you open VE on mobile and want to edit an existing reference, you need to tap it twice." which remains a bug.
Comment 5 James Forrester 2014-06-26 18:10:38 UTC
Looks like it's fixed by bug 67063's fix.
Comment 6 Rummana Yasmeen 2014-06-27 19:06:09 UTC
This bug did not get resolved by Ed's fix.

The issue here is, the first tap is showing a lighter highlight which is equivalent to the hover effect in the desktop and for the second tap it is actually selecting that node.

This is why, even if you have selected a reference (with two tap) , you can still semi-select another node (with lighter highlight)at the same time  

For mobile, it should require one single tap to select any reference/image etc.
Comment 7 Gerrit Notification Bot 2014-06-28 21:55:40 UTC
Change 142839 had a related patch set uploaded by Esanders:
Select highlights on touchend event

https://gerrit.wikimedia.org/r/142839
Comment 8 Gerrit Notification Bot 2014-06-30 18:20:14 UTC
Change 142839 merged by jenkins-bot:
Select highlights on touchend event

https://gerrit.wikimedia.org/r/142839
Comment 9 Rummana Yasmeen 2014-06-30 20:26:46 UTC
Verified the fix in Betalabs.
Comment 10 Rummana Yasmeen 2014-07-08 00:25:21 UTC
Verified the fix in test2
Comment 11 Rummana Yasmeen 2014-07-10 20:19:50 UTC
Verified the fix in production

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links