View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
09932Bug reportsResponse browsingpublic2015-12-09 15:56
ReporterDenisChenu Assigned ToLouisGac 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.50.x 
Summary09932: review response browsing to be usuable
Description

In 2.06/master : browsetable take all window : scrolling the window scroll the table. Actually : browse table have is own "scrollbale" div.

Steps To Reproduce

See pic to see difference

  • We have 2 scroll for the table : one for window, and for the div
  • We can not scroll right/left on table if we don't scroll windo before
  • We don't see page number and move next/prev when scrolling (or not)
Additional Information

I think we mus hide the "General" option : it's not used when we go to "Token table" or "Response table"

In 1.92 : i do some test with scrollable div inside the windows : Carsten revert it.

TagsNo tags attached.
Attached Files
206-1.png (158,062 bytes)
250-1.png (157,422 bytes)
250-2.png (174,096 bytes)
survey_archive_10000.lsa (19,498 bytes)
quick-fix.png (78,690 bytes)   
quick-fix.png (78,690 bytes)   
quick-fix2.png (78,570 bytes)   
quick-fix2.png (78,570 bytes)   
Bug heat6
Complete LimeSurvey version number (& build)150930
I will donate to the project if issue is resolvedNo
Browserfirefox
Database type & versionnot relevant
Server OS (if known)not relevant
Webserver software & version (if known)not relevant
PHP Versionnot relevant

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2015-09-30 17:56

developer   ~33264

Last edited: 2015-09-30 17:56

This clearly seens with a big survey with a lot of answer then upload a survey

Browse response is a lot used by our user's.

DenisChenu

DenisChenu

2015-09-30 17:57

developer   ~33265

@LouisGac : the "scrolling is done on the DIV. In 2.05 : scroll windows scroll the table (and only the table)

DenisChenu

DenisChenu

2015-09-30 18:25

developer   ~33266

Last edited: 2015-09-30 18:27

Quick fix (need more width ...) done with firebug:

#gbox_displayresponses{max-width:100%}
#gview_displayresponses{max-width:100%}
#displayresponses_toppager{max-width:100%}
#pager{max-width:100%}

and remove the overflow on .col-lg-12.content-right

Javascript still here to move top pager and bottom pager right/left (https://github.com/LimeSurvey/LimeSurvey/blob/master/scripts/admin/listresponse.js#L13)

HTML bug : we have 2 #pager (one for top, one for bottom)

LouisGac

LouisGac

2015-10-01 12:23

developer   ~33271

we tried it on different browsers and OS, we can't reproduce the bug.
everybody has the scrollable div.

DenisChenu

DenisChenu

2015-10-01 12:35

developer   ~33272

You don't look at the pic :
I have the scrollable duv BUT : in 2.05 : we have NOT a scrolabke div.

Scroll window directly scroll the response table. No double scroll : one for div+one for window.

My labtop have a height of 768, then i don't have the scroll of the div if i don't scroll the window before

DenisChenu

DenisChenu

2015-10-01 12:36

developer   ~33273

And look at the 2 fix picture : only one scroll : only window scroll ....

LouisGac

LouisGac

2015-10-01 13:03

developer   ~33274

Last edited: 2015-10-01 13:11

ok so that's just not a bug.
this is the wanted feature.
We can't provide support for 1024x768, 800x600 (whatever, most of the template will work on a 1024x768, 800x600)

DenisChenu

DenisChenu

2015-10-01 14:06

developer   ~33275

Not true : it's a bug ....

In 2.50 : browse response table is really unusuable

Usuability is not a feature ....

LouisGac

LouisGac

2015-10-01 14:30

developer   ~33276

I just spoke about it with Carsten, and only 4% of people are using a resolution of 1024x768 in the world, probably less for limesurvey users... so it's not a huge priority.

Whatever, the website itself should not be horizontal scrollable. There is a lot of studies about it, and users hate horinzontal scroll :
http://www.nngroup.com/articles/scrolling-and-scrollbars/

What I'll do in the future, is providing the possibility to display the table in a popup windows, so this popup windows, and only this one, will be horizontal-scrollable.

DenisChenu

DenisChenu

2015-10-01 15:07

developer   ~33277

Last edited: 2015-10-01 15:42

I test with : zoom less x4 and : same thing.

It's not only a height question. It's usuability.

Here : table must have a scroll system bar if : it's to wide or tool long (remine user can want to view 100 response for exemple).

This scroll bar for usuability must be set to window scroll bar and no where else.

Don't tell me the usuability of 2.50 browse table is better than 2.06 system.

Ps : "4% of people are using a resolution of 1024x768" where did you found that ? I have a lot of client using limesurvey with more little screen than this one. Using LS in train on a little portable computer.

DenisChenu

DenisChenu

2015-10-06 11:32

developer   ~33320

See old issue and old revert:
https://github.com/Shnoulle/LimeSurvey-shnoulle/commit/35c16aaafa406686f23e290c78b58b08060adfc0#diff-b9d2eff036d453b4d5d16187a490d020R557

https://github.com/Shnoulle/LimeSurvey-shnoulle/commit/dd475c8051cebd0493a83fa6b9a68a8d97c8b7fe#diff-b9d2eff036d453b4d5d16187a490d020L573

Comment by Carsten : "Dev Made the response browsing table using the browser scrollbars again, because usability bad with bigger result sets."

c_schmitz

c_schmitz

2015-10-13 13:53

administrator   ~33335

Denis: See http://www.w3schools.com/browsers/browsers_display.asp

I agree that we should revert the grid to use normal scrollbars again.

DenisChenu

DenisChenu

2015-10-13 14:04

developer   ~33336

Last edited: 2015-10-13 14:18

It's not only a width issue, but a high issue . Some user ask to see 100 line at a time. And with 100 lines: no scrollbars with a big screen ....

Nut i can fix it in a plugin/modules ....

PS: and the "jquery" tool is show always in 2.06 , show at ..... right, right, right again in 2.50 (Page X on Y/ next/previous)
PS2: overflow : auto on the data : see http://stackoverflow.com/questions/4984969/does-jquery-jqgrid-support-horizontal-scroll-bar-and-frozen-columns

Issue History

Date Modified Username Field Change
2015-09-30 17:54 DenisChenu New Issue
2015-09-30 17:54 DenisChenu File Added: 206-1.png
2015-09-30 17:54 DenisChenu File Added: 250-1.png
2015-09-30 17:54 DenisChenu File Added: 250-2.png
2015-09-30 17:55 DenisChenu File Added: survey_archive_10000.lsa
2015-09-30 17:56 DenisChenu Note Added: 33264
2015-09-30 17:56 DenisChenu Note Edited: 33264
2015-09-30 17:57 DenisChenu Note Added: 33265
2015-09-30 17:57 DenisChenu Assigned To => LouisGac
2015-09-30 17:57 DenisChenu Status new => assigned
2015-09-30 17:58 DenisChenu Summary revie response browsing to be usuable => review response browsing to be usuable
2015-09-30 17:58 DenisChenu Additional Information Updated
2015-09-30 18:25 DenisChenu Note Added: 33266
2015-09-30 18:25 DenisChenu File Added: quick-fix.png
2015-09-30 18:26 DenisChenu File Added: quick-fix2.png
2015-09-30 18:27 DenisChenu Note Edited: 33266
2015-10-01 12:23 LouisGac Note Added: 33271
2015-10-01 12:35 DenisChenu Note Added: 33272
2015-10-01 12:36 DenisChenu Note Added: 33273
2015-10-01 13:03 LouisGac Note Added: 33274
2015-10-01 13:11 LouisGac Note Edited: 33274
2015-10-01 14:06 DenisChenu Note Added: 33275
2015-10-01 14:30 LouisGac Note Added: 33276
2015-10-01 15:07 DenisChenu Note Added: 33277
2015-10-01 15:42 DenisChenu Note Edited: 33277
2015-10-06 11:32 DenisChenu Note Added: 33320
2015-10-13 13:53 c_schmitz Note Added: 33335
2015-10-13 14:04 DenisChenu Note Added: 33336
2015-10-13 14:10 DenisChenu Note Edited: 33336
2015-10-13 14:18 DenisChenu Note Edited: 33336
2015-12-09 15:56 LouisGac Status assigned => closed
2015-12-09 15:56 LouisGac Resolution open => fixed