| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 11852 | Bug reports | Survey editing | public | 2016-11-01 14:47 | 2017-07-21 17:08 |
| Reporter | tammo | Assigned To | DenisChenu | ||
| Priority | none | Severity | partial_block | ||
| Status | closed | Resolution | fixed | ||
| Summary | 11852: Slider questions messed up due to wrong statement in survey.css | ||||
| Description | See image. In the html the tag <div class="ls-input-group"> appears. Directly before that tab and after that tag there are col-xx-xx statements, resulting in nested cols. I think the tag should be: <div class="ls-input-group row"> and in the css, the statemnent: display: table should be changed. | ||||
| Tags | answers_html | ||||
| Bug heat | 6 | ||||
| Complete LimeSurvey version number (& build) | 2.55 git | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | ff | ||||
| Database type & version | ff | ||||
| Server OS (if known) | ff | ||||
| Webserver software & version (if known) | ff | ||||
| PHP Version | ff | ||||
 |
You don't have a reset here : with reset : need a display:table. I'm unsure about "ls-input-group" : i use it more like input-group + input-group-addon in BS : great to add a little information just right/left an input. Must be used only for suffix/preffix, but here : there are answers part inside. For the screenshot : what is the 'column_width' setting ? |
|
|
Proposed solution: Include reset IN slider-container : move the ls-input-group to slider container. |
 |
This is the survey I use for testing |
|
|
Display: all in one... |
|
|
OK, thanks : yes big issue here :) |
|
|
Fix committed to answers_html branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=21226 |
|
|
OK : fixed : put input-group in the slider only completly fix the issue. reset button is just near slider : more clear. But right answer part is after reset button. About ls-input-group .ls-input-group-addon : see https://github.com/LimeSurvey/LimeSurvey/commit/a0e26df9789c68639ab1179334f35c058d6fa787#diff-9ad3040bab7d1b1a562414c5f7d2edebR98 ls-input-group-addon must have near same purpose than http://getbootstrap.com/components/#input-groups-basic i think. And, for example Your vanity URL is not good on little phone (no place in input) |
|
|
I think we are getting close, but not there yet. Just an idea: currently the order of divs is: prefix - slider - suffix Could this be an idea: prefix - suffix - slider using col-sm-push/pull http://getbootstrap.com/css/#grid-column-ordering slider-2.png (25,675 bytes) |
|
|
Maybe great ide : bnut why ? What is the purpose ? |
|
|
I suggested this to avoid that in mobile it looks like on the picture. I think it might be better to have the prefix and suffix both above the slider and not like now: prefix above, suffix below slider... |
|
|
More like this: see image 3 slider-3.png (46,586 bytes) |
|
|
Where does that red dotted line come from? |
|
|
See image 4 for better alignment: all close to red dotted line. |
|
|
Image 4 slider-5.png (13,243 bytes) |
|
|
OK, understand, but still : i'm not OK with this. HTML must be presented like he must be readed, not like it must be seen. If you put an order : you didn't put it for 'visually' see in thhis order : you put an order because it's better. And think here : its' more a template related issue : i find a way for 'lumen' but not for devel. For the red border : it's here to be sure ls-answers have all same margin/padding (row fixing). It's devel template : |
|
|
Fix committed to answers_html branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=21249 |
 |
Version 3.0.0-beta.1 released |
