View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
18648 | Bug reports | Other | public | 2023-02-22 11:57 | 2023-10-16 08:41 |
Reporter | 2BITS_PL | Assigned To | tibor.pacalat | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 5.6.x | ||||
Summary | 18648: Unable to upload a file with theme animations enabled | ||||
Description | The Participant cannot upload files when the Fruity theme is set for the survey with the "Animation Body" feature enabled. Also confirmed on demo.limesurvey.org (v5.6.6) | ||||
Steps To Reproduce | Steps to reproduceAdd a question like "Upload Files" to your survey. Proceed to completing the survey by clicking the Upload file button. Expected resultIt should bring up a modal window with the option to upload a file Actual resultA modal window opens, but it is under a gray layer and there is no option to add files. | ||||
Tags | No tags attached. | ||||
Bug heat | 14 | ||||
Complete LimeSurvey version number (& build) | Version 5.4.11+221114 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | SQL Server 2019 | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | v8.0.27 NTS x64 | ||||
Can you please click F12 in the browser and give us a screenshot of any errors shown in the JavaScript console? Thank you. |
|
I have no JS errors locally. And on the demo it shows something: If you change the "Animation body" to "No", then it works fine. In my opinion, the point is that animated elements in CSS are embedded later. A simple example, with the Animated Body option enabled, open a survey, open the DevToll in it and for the #outerframeContainer element and disable "animation-name", and everything will be back to normal, except that the element will not be animated. |
|
I need information if and when the fix will be deployed. |
|
@tibor.pacalat, if you have time to check later. |
|
I can confirm that this is happening. |
|
When this is tested, lot of transformations should be tested as to review they are not impacted. |
|
Need the reason why it was added maybe to test ? default is none |
|
"None" it was not added, is the new value after update. |
|
Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35557 |
|
Tested and merged. |
|
Does this happen on v6? |
|
@gabrieljenik Yes it does. Can we agree on protocol, who should check which versions are affected? It is really bad the way we do it at the moment, reopening closed tickets because this was not checked before handing it further. |
|
I think the best is that you decide which versions should tickets be applied to. v5, v6, v5+6, ...
Tickets turn to resolved automatically, is not done manually. |
|
Ok, I'll try to do that. I usually put tickets to resolved after I merge PRs attached to the ticket in. |
|
OK, then I think we should wait until all versions are good for putting it on resolved. |
|
Fixed in Release 5.6.40+231005 |
|
@tibor.pacalat Does master also requires the fix? |
|
@gabrieljenik yes it does. |
|
Porting it |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35607 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=35608 |
|
Tested and merged. |
|
Fixed in Release 6.3.0+231016 |
|
LimeSurvey: 5.x 4acd0478 2023-09-28 16:03 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 18648: Unable to upload a file with theme animations enabled (#3356) Co-authored-by: lapiudevgit <devgit@lapiu.biz> |
Affected Issues 18648 |
|
mod - themes/survey/fruity/views/subviews/header/custom_header.twig | Diff File | ||
LimeSurvey: master 69efa21e 2023-10-12 16:15 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 18648: Unable to upload a file with theme animations enabled (03529) |
Affected Issues 18648 |
|
mod - themes/survey/fruity/views/subviews/header/custom_header.twig | Diff File | ||
LimeSurvey: master 69efa21e 2023-10-12 16:15 Gabriel Jenik Committer: GitHub Details Diff |
Fixed issue 18648: Unable to upload a file with theme animations enabled (03529) |
Affected Issues 18648 |
|
mod - themes/survey/fruity/views/subviews/header/custom_header.twig | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2023-02-22 11:57 | 2BITS_PL | New Issue | |
2023-02-22 12:00 | ollehar | Note Added: 73930 | |
2023-02-22 12:00 | ollehar | Bug heat | 0 => 2 |
2023-02-22 15:57 | 2BITS_PL | Note Added: 73936 | |
2023-02-22 15:57 | 2BITS_PL | Bug heat | 2 => 4 |
2023-03-09 14:51 | gabrieljenik | Status | new => acknowledged |
2023-03-14 15:25 | gabrieljenik | Status | acknowledged => confirmed |
2023-08-04 14:53 | 2BITS_PL | Note Added: 76434 | |
2023-08-07 13:35 | ollehar | Assigned To | => tibor.pacalat |
2023-08-07 13:35 | ollehar | Status | confirmed => assigned |
2023-08-07 13:36 | ollehar | Note Added: 76453 | |
2023-08-07 16:58 | gabrieljenik | Severity | block => partial_block |
2023-08-07 18:11 | tibor.pacalat | Status | assigned => confirmed |
2023-08-07 18:11 | tibor.pacalat | Note Added: 76465 | |
2023-08-07 18:11 | tibor.pacalat | Bug heat | 4 => 6 |
2023-08-08 23:53 | gabrieljenik | Assigned To | tibor.pacalat => gabrieljenik |
2023-08-15 16:28 | gabrieljenik | Assigned To | gabrieljenik => DenisChenu |
2023-08-15 16:28 | gabrieljenik | Status | confirmed => ready for code review |
2023-08-15 16:28 | gabrieljenik | Note Added: 76601 | |
2023-08-15 16:28 | gabrieljenik | Bug heat | 6 => 8 |
2023-08-15 16:28 | gabrieljenik | Note Added: 76602 | |
2023-09-07 18:28 | DenisChenu | Assigned To | DenisChenu => gabrieljenik |
2023-09-07 18:28 | DenisChenu | Status | ready for code review => ready for testing |
2023-09-07 18:28 | DenisChenu | Note Added: 76905 | |
2023-09-07 18:28 | DenisChenu | Bug heat | 8 => 10 |
2023-09-07 18:36 | gabrieljenik | Note Added: 76908 | |
2023-09-25 20:17 | gabrieljenik | Assigned To | gabrieljenik => tibor.pacalat |
2023-09-25 20:17 | gabrieljenik | Status | ready for testing => ready for merge |
2023-09-28 14:03 | Changeset attached | => LimeSurvey 5.x 4acd0478 | |
2023-09-28 14:03 | guest | Note Added: 77395 | |
2023-09-28 14:03 | guest | Bug heat | 10 => 12 |
2023-09-28 14:03 | tibor.pacalat | Status | ready for merge => resolved |
2023-09-28 14:03 | tibor.pacalat | Resolution | open => fixed |
2023-09-28 14:03 | tibor.pacalat | Note Added: 77396 | |
2023-09-28 16:44 | gabrieljenik | Note Added: 77409 | |
2023-09-28 16:45 | gabrieljenik | Status | resolved => feedback |
2023-09-28 16:45 | gabrieljenik | Resolution | fixed => reopened |
2023-09-28 17:09 | tibor.pacalat | Note Added: 77412 | |
2023-09-28 17:59 | gabrieljenik | Note Added: 77420 | |
2023-09-28 18:04 | tibor.pacalat | Note Added: 77423 | |
2023-09-28 18:12 | gabrieljenik | Note Added: 77425 | |
2023-10-04 11:30 | LimeBot | Note Added: 77501 | |
2023-10-04 11:30 | LimeBot | Status | feedback => closed |
2023-10-04 11:30 | LimeBot | Resolution | reopened => fixed |
2023-10-04 11:30 | LimeBot | Bug heat | 12 => 14 |
2023-10-05 00:04 | gabrieljenik | Note Added: 77513 | |
2023-10-05 00:04 | gabrieljenik | Status | closed => feedback |
2023-10-05 00:04 | gabrieljenik | Resolution | fixed => reopened |
2023-10-05 11:46 | tibor.pacalat | Note Added: 77516 | |
2023-10-05 11:46 | tibor.pacalat | Assigned To | tibor.pacalat => gabrieljenik |
2023-10-05 15:49 | gabrieljenik | Status | feedback => assigned |
2023-10-05 15:49 | gabrieljenik | Note Added: 77522 | |
2023-10-11 01:06 | gabrieljenik | Status | assigned => ready for code review |
2023-10-11 01:06 | gabrieljenik | Note Added: 77616 | |
2023-10-11 01:06 | gabrieljenik | Assigned To | gabrieljenik => DenisChenu |
2023-10-12 11:11 | DenisChenu | Assigned To | DenisChenu => tibor.pacalat |
2023-10-12 11:11 | DenisChenu | Status | ready for code review => ready for testing |
2023-10-12 14:15 | Changeset attached | => LimeSurvey master 69efa21e | |
2023-10-12 14:15 | Changeset attached | => LimeSurvey master 69efa21e | |
2023-10-12 14:15 | guest | Note Added: 77661 | |
2023-10-12 14:15 | guest | Note Added: 77662 | |
2023-10-12 14:16 | tibor.pacalat | Status | ready for testing => resolved |
2023-10-12 14:16 | tibor.pacalat | Note Added: 77663 | |
2023-10-16 08:41 | LimeBot | Note Added: 77723 | |
2023-10-16 08:41 | LimeBot | Status | resolved => closed |
2023-10-16 08:41 | LimeBot | Resolution | reopened => fixed |