View Issue Details

This bug affects 1 person(s).
 8
IDProjectCategoryView StatusLast Update
19188Bug reportsUsability/user experiencepublic2023-10-30 14:00
Reporterak125199 Assigned Tomfavetti  
PrioritynoneSeveritytrivial 
Status closedResolutionfixed 
Product Version6.3.x 
Summary19188: css problems in survey permissions after reload
Description

when reloading the survey permissions site in a survey both dropdown fields are loosing their width and overlapping

Steps To Reproduce

Steps to reproduce

open or create a survey and goto survey permissions. then reload site (F2).

Expected result

all content should remain the same

Actual result

both dropdown elements are loosing their width and are overlapping, as shown in the screenshot

TagsNo tags attached.
Attached Files
Bug heat8
Complete LimeSurvey version number (& build)6.3.1
I will donate to the project if issue is resolvedNo
Browser
Database type & versionpsql 14
Server OS (if known)rocky 9.2
Webserver software & version (if known)
PHP Versionphp 8.1

Users monitoring this issue

There are no users monitoring this issue.

Activities

mfavetti

mfavetti

2023-10-24 10:54

developer   ~77903

this is an issue with select2 library

mfavetti

mfavetti

2023-10-24 11:14

developer   ~77904

ok i have a potential solution, but it affects MANY files....

there must be some difference between the computed width when loading from sidebar and loading from direct URL (maybe sidebar is not replaced, so width is less?) anyways setting width to 100% for select2 fixes the issue and renders the same in both situations. HOWEVER, this also affects other instances of select-search that use adminbasics... potentially breaking other stuff. could need a lot of testing

mfavetti

mfavetti

2023-10-24 11:28

developer   ~77905

okay I tested through them and it doesn't seem to cause any adverse affects.... also found one use of select2 (application\views\admin\survey\subview\accordion_generaloptions_panel.php) that already explicitly set width: 100%, implying they came across the same solution, but didn't apply to all cases

mfavetti

mfavetti

2023-10-24 11:31

developer   ~77906

does not affect 5.x

mfavetti

mfavetti

2023-10-24 11:32

developer   ~77907

master: https://github.com/LimeSurvey/LimeSurvey/pull/3569

DenisChenu

DenisChenu

2023-10-24 11:57

developer   ~77908

Upset about set 100% for all other select2 too.

mfavetti

mfavetti

2023-10-25 00:01

developer   ~77945

For tester to reproduce issue:

mfavetti

mfavetti

2023-10-25 00:01

developer   ~77946

After:

DenisChenu

DenisChenu

2023-10-25 08:42

developer   ~77956

this is an issue with select2 library

But why when reload ?

mfavetti

mfavetti

2023-10-25 08:53

developer   ~77957

yeah I'm not sure why either. I guessed that there is some difference between clicking the sidebar link and loading the route directly.

mfavetti

mfavetti

2023-10-25 11:22

developer   ~77966

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35665

tibor.pacalat

tibor.pacalat

2023-10-25 11:22

administrator   ~77967

Tested and merged.

LimeBot

LimeBot

2023-10-30 14:00

administrator   ~78135

Fixed in Release 6.3.2+231031

Related Changesets

LimeSurvey: master 347e9a74

2023-10-25 13:22

mfavetti

Committer: GitHub


Details Diff
Fixed issue 19188: css problems in survey permissions after reload (03569) Affected Issues
19188
mod - application/views/surveyPermissions/index.php Diff File

Issue History

Date Modified Username Field Change
2023-10-23 13:49 ak125199 New Issue
2023-10-23 13:49 ak125199 File Added: Screenshot 2023-10-23 134616.png
2023-10-24 02:15 mfavetti Assigned To => mfavetti
2023-10-24 02:15 mfavetti Status new => assigned
2023-10-24 10:54 mfavetti Note Added: 77903
2023-10-24 10:54 mfavetti Bug heat 0 => 2
2023-10-24 11:14 mfavetti Note Added: 77904
2023-10-24 11:28 mfavetti Note Added: 77905
2023-10-24 11:31 mfavetti Note Added: 77906
2023-10-24 11:32 mfavetti Note Added: 77907
2023-10-24 11:32 mfavetti Assigned To mfavetti => DenisChenu
2023-10-24 11:32 mfavetti Status assigned => ready for code review
2023-10-24 11:57 DenisChenu Status ready for code review => in code review
2023-10-24 11:57 DenisChenu Note Added: 77908
2023-10-24 11:57 DenisChenu Bug heat 2 => 4
2023-10-24 15:56 DenisChenu Assigned To DenisChenu => mfavetti
2023-10-25 00:01 mfavetti Note Added: 77945
2023-10-25 00:01 mfavetti File Added: LimeSurvey - Google Chrome 2023-10-24 14-59-37.mp4
2023-10-25 00:01 mfavetti Note Added: 77946
2023-10-25 00:01 mfavetti File Added: LimeSurvey - Google Chrome 2023-10-24 15-00-37.mp4
2023-10-25 08:42 DenisChenu Note Added: 77956
2023-10-25 08:53 mfavetti Note Added: 77957
2023-10-25 09:44 DenisChenu Assigned To mfavetti => tibor.pacalat
2023-10-25 09:44 DenisChenu Status in code review => ready for testing
2023-10-25 11:22 mfavetti Changeset attached => LimeSurvey master 347e9a74
2023-10-25 11:22 mfavetti Note Added: 77966
2023-10-25 11:22 mfavetti Assigned To tibor.pacalat => mfavetti
2023-10-25 11:22 mfavetti Resolution open => fixed
2023-10-25 11:22 tibor.pacalat Status ready for testing => resolved
2023-10-25 11:22 tibor.pacalat Note Added: 77967
2023-10-25 11:22 tibor.pacalat Bug heat 4 => 6
2023-10-30 14:00 LimeBot Note Added: 78135
2023-10-30 14:00 LimeBot Status resolved => closed
2023-10-30 14:00 LimeBot Bug heat 6 => 8