Opened 7 years ago

Last modified 7 years ago

#15623 closed defect

dojox.mobile.IconContainer: misbehavior in editing mode with scrollable view — at Version 4

Reported by: Adrian Vasiliu Owned by: Eric Durocher
Priority: high Milestone: 1.8
Component: DojoX Mobile Version: 1.8.0b1
Keywords: Cc: cjolif
Blocked By: Blocking:

Description (last modified by cjolif)

When IconContainer is contained in a dojox.mobile.ScrollableView, the editing operation misbehaves as soon as the view is scrolled.

How to reproduce:

  1. Launch dojox/mobile/tests/test_IconContainer-pubsub.html

Dojo 1.8b1

(reproduced on iPhone 4S iOS 5.0.1 and Chrome 19).

  1. Press the "Start Edit" button (or do a long touch on an icon).
  1. As long as the view is not scrolled, the behavior is the expected one: when touching an icon, it zooms a bit while staying at the same location. So far so good.
  1. Now scroll the view (on desktop, you need to reduce the size of the window for this purpose), and try again to touch an icon.

==> KO. Now the icon "jumps" to a different location, possibly outside of the visible area. The distance from the expected position is equal to the amount of scroll. This makes the editing operation hard or impossible to use for scrolled views.

Change History (5)

comment:1 Changed 7 years ago by Adrian Vasiliu

I have tested that browser scroll is NOT an issue (that is, the bug does not hurt when the IconContainer is inside, say, a dojox/mobile/View). The trouble was only with our scrollable view.

Last edited 7 years ago by Adrian Vasiliu (previous) (diff)

comment:2 Changed 7 years ago by cjolif

Adrian, are you sure about:

if(scroll){ // this node is placed inside a scrollable
	var pos = enclosingScrollable.getPos();
	dx = pos.x;
	dy = pos.y;
}

It does not look like scrollable is defined?

Changed 7 years ago by Adrian Vasiliu

Attachment: patch15623.patch added

Fix misbehavior in editing mode inside ScrollableView - Adrian Vasiliu, IBM, CCLA

comment:3 Changed 7 years ago by Adrian Vasiliu

Argh. Reattached the correct patch. Thanks.

comment:4 Changed 7 years ago by cjolif

Description: modified (diff)
Version: 1.7.31.8.0b1
Note: See TracTickets for help on using tickets.