New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.1.0-rc.1 showPageLoadingMsg demo does not work for android 2.3.3 #3786

Closed
bmurmistro opened this Issue Mar 9, 2012 · 9 comments

Comments

Projects
None yet
4 participants
@bmurmistro

bmurmistro commented Mar 9, 2012

Tried this on an emulator and my phone without success:
http://jquerymobile.com/demos/1.1.0-rc.1/docs/config/loadingMessageTextVisible.html
I upgraded my app to 1.1.0-rc.1 and some of my showPageLoadingMsg calls do work in android 2.3.3, but most do not. I haven't tried any other versions of android.

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Mar 10, 2012

Contributor

@bmurmistro

Are you sure it's just not showing up farther down the page? I had that issue when I was playing around with it. I'm actually only able to reproduce on the latest:

http://jquerymobile.com/test/docs/config/loadingMessageTextVisible.html

Obviously still an issue.

Contributor

johnbender commented Mar 10, 2012

@bmurmistro

Are you sure it's just not showing up farther down the page? I had that issue when I was playing around with it. I'm actually only able to reproduce on the latest:

http://jquerymobile.com/test/docs/config/loadingMessageTextVisible.html

Obviously still an issue.

@johnbender johnbender closed this Mar 10, 2012

@johnbender johnbender reopened this Mar 10, 2012

@bmurmistro

This comment has been minimized.

Show comment
Hide comment
@bmurmistro

bmurmistro Mar 10, 2012

Hi John,

I just tried it again and if I scroll down I do see it on the bottom. I could have sworn it wasn't working at all earlier so I did a little more testing.

Android 2.1 (emulator) - Seemed to work as expected
Android 2.2 (emulator) - Did not work. I did a refresh and it worked fine (including the location of the spinner)
Android 2.3 (htc mytouch) - Tried it again and it didn't work. I did a refresh and it worked fine. (the location of the spinner was at the bottom)
Kindle Fire - Did not work. I did a refresh and it worked fine (including the location of the spinner)

It appears that we have two issues here:

Location of the spinner in 2.3
Loader not showing up in 2.2 and 2.3 ( initial load ... does appear to be somewhat random )

Hope this helps!

bmurmistro commented Mar 10, 2012

Hi John,

I just tried it again and if I scroll down I do see it on the bottom. I could have sworn it wasn't working at all earlier so I did a little more testing.

Android 2.1 (emulator) - Seemed to work as expected
Android 2.2 (emulator) - Did not work. I did a refresh and it worked fine (including the location of the spinner)
Android 2.3 (htc mytouch) - Tried it again and it didn't work. I did a refresh and it worked fine. (the location of the spinner was at the bottom)
Kindle Fire - Did not work. I did a refresh and it worked fine (including the location of the spinner)

It appears that we have two issues here:

Location of the spinner in 2.3
Loader not showing up in 2.2 and 2.3 ( initial load ... does appear to be somewhat random )

Hope this helps!

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Mar 11, 2012

Contributor

@bmurmistro

It does indeed.

Contributor

johnbender commented Mar 11, 2012

@bmurmistro

It does indeed.

@benjaminthatcher

This comment has been minimized.

Show comment
Hide comment
@benjaminthatcher

benjaminthatcher Mar 11, 2012

I think this issue may be related:
#3552

benjaminthatcher commented Mar 11, 2012

I think this issue may be related:
#3552

@ghost ghost assigned johnbender Mar 15, 2012

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Mar 15, 2012

Contributor

[edit] useless

Contributor

johnbender commented Mar 15, 2012

[edit] useless

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Mar 15, 2012

Contributor

All,

You can try out my proposed fix by including:

<script src="http://jquerymobile.com/branches/3786-fix/js/"></script>

for jquery mobile

@scottjehl

15b1ce6

That fixes at least part of the issue, and also might speed up our scrolling a bit but really requires your review just to be sure. My grasp of possible height calculation issues is very weak.

Contributor

johnbender commented Mar 15, 2012

All,

You can try out my proposed fix by including:

<script src="http://jquerymobile.com/branches/3786-fix/js/"></script>

for jquery mobile

@scottjehl

15b1ce6

That fixes at least part of the issue, and also might speed up our scrolling a bit but really requires your review just to be sure. My grasp of possible height calculation issues is very weak.

@johnbender

This comment has been minimized.

Show comment
Hide comment
@johnbender

johnbender Mar 15, 2012

Contributor

All,

You can also test on your android devices by hitting:

http://jquerymobile.com/branches/3786-fix/docs/config/loadingMessageTextVisible.html

Contributor

johnbender commented Mar 15, 2012

All,

You can also test on your android devices by hitting:

http://jquerymobile.com/branches/3786-fix/docs/config/loadingMessageTextVisible.html

@bmurmistro

This comment has been minimized.

Show comment
Hide comment
@scottjehl

This comment has been minimized.

Show comment
Hide comment
@scottjehl

scottjehl Mar 20, 2012

Contributor

Looks good to me!

On Mar 20, 2012, at 8:00 PM, bmurmistro wrote:

http://jquerymobile.com/branches/3786-fix/docs/config/loadingMessageTextVisible.html looks good.


Reply to this email directly or view it on GitHub:
#3786 (comment)

Contributor

scottjehl commented Mar 20, 2012

Looks good to me!

On Mar 20, 2012, at 8:00 PM, bmurmistro wrote:

http://jquerymobile.com/branches/3786-fix/docs/config/loadingMessageTextVisible.html looks good.


Reply to this email directly or view it on GitHub:
#3786 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment