Opened 7 years ago

Closed 4 years ago

#15610 closed defect (patchwelcome)

Carousel page indicator might be desynchronized on Android 4.0

Reported by: Damien Mandrioli Owned by: Patrick Ruzand
Priority: undecided Milestone: 1.13
Component: DojoX Mobile Version: 1.8.0b1
Keywords: Cc:
Blocked By: Blocking:

Description

Test case: dojox/mobile/tests/test_Carousel.html

Sometimes, when changing page using top navigation buttons, the page indicator (3 little dots) is not updated. See attached images.

It seems this occurs only on Android 4.0.

When the indicator is not updated, touching the screen solve the problem.

Attachments (3)

normal.png (41.8 KB) - added by Damien Mandrioli 7 years ago.
abnormal.png (42.0 KB) - added by Damien Mandrioli 7 years ago.
ticket15610.patch (2.6 KB) - added by Sebastien Brunot 6 years ago.
fix based on the fix for #15988: doesn't work in landscape mode… (IBM CCLA)

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by Damien Mandrioli

Attachment: normal.png added

Changed 7 years ago by Damien Mandrioli

Attachment: abnormal.png added

comment:1 Changed 7 years ago by Damien Mandrioli

Also reproducible with:

  • dojox/tests/test_DataCarousel-demo.html
  • dojox/tests/test_DataCarousel.html

comment:2 Changed 7 years ago by Damien Mandrioli

Same kind of problem with dojox/mobile/tests/test_TabBar-syncWithViews.html

comment:3 Changed 6 years ago by Sebastien Brunot

The issue is with the WebView / stock browser not repainting its content after styling a node (see also this discussion on envoyjs forum: http://forums.enyojs.com/discussion/433/android-browser-does-not-re-render-ui-till-touch). I've written a first patch based on what was done for #15988, but it doesn't work when using the device on landscape mode... We need to find a reliable way to implement a _forceRendering method for android...

Changed 6 years ago by Sebastien Brunot

Attachment: ticket15610.patch added

fix based on the fix for #15988: doesn't work in landscape mode… (IBM CCLA)

comment:4 Changed 6 years ago by Patrick Ruzand

Owner: changed from Eric Durocher to Patrick Ruzand
Status: newassigned

comment:5 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 3+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.