nikhilbchilwant / google-web-toolkit-incubator

Automatically exported from code.google.com/p/google-web-toolkit-incubator
1 stars 1 forks source link

Paging Options hard codes the scrollTableLoading.gif image instead of using the GWT.getModuleBaseURL() #271

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of gwt and gwt-incubator are you using?
GWT 1.6

What OS and browser are you using?
Windows XP, IE, Chrome and Firefox all versions

Do you see this error in hosted mode, web mode, or both?
Both

(If possible, please include a test case that shows the problem)

PagingScrollTable with PagingOptions is reporting "scrollTableLoading.gif" 
missing.

Hopefully using the test case you have generously provided, what steps will 
reproduce the problem? 
1.

Implement a grid using the PagingScrollTable with PagingOptions. When the 
user clicks on page next/previous, the paging options displays an page load 
indicator "scrollTableLoading.gif" while the data being loaded (Generally 
you don't see this image being displayed next to the page next/previous as 
the request rows loads fast). If you put a delay then you will see that the  
scollTableImage.gif missing icon displayed by the browser.

2.
3.

What is the expected output? What do you see instead?

scrollTableLoading.gif "spinner" image should be displayed. Currently I see 
a missing image icon. 

Workaround if you have one:

None. 

However, it is a minor fix in PagingOptions.java in the 
com.google.gwt.gen2.table.client package.

    loadingImage = new Image(GWT.getModuleBaseURL() + 
"scrollTableLoading.gif");

Please provide any additional information below,  and thank you for taking 
the time and effort to report this issue, as good issue reports are 
critical for our quest to make GWT's new widgets and libraries shine.

Original issue reported on code.google.com by rtsel...@gmail.com on 28 Apr 2009 at 11:22

GoogleCodeExporter commented 9 years ago
Fixed in r1682.

Original comment by itruett on 18 Jun 2009 at 8:58