Project

General

Profile

Actions

Bug #80416

closed

Imageorient icons not working with slow Internet connection

Added by Tabea Oertel about 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend JavaScript
Target version:
-
Start date:
2017-03-23
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When opening an image or text&images content element with a slow connection, the icons below the dropdown of the imageorient selection are shown but do not work. When clicked the selection dont get applied.
The dropdown works properly.

The reason is located in the javascript file typo3_src/typo3/sysext/backend/Resources/Public/JavaScript/FormEngine/Element/SelectSingleElement.js
The icons are used without checking beforehand if they are already loaded. So the icons do not get the onClick event when you have a slow connection.

You can check the behavior in chrome using the developer tools. In the network tab you can use the throttling functionality.
In my case the problem occurred with 'Regular 3G'.

Used TYPO3 Version is 7.6

Imageorient


Files

image-orient.jpg (17.6 KB) image-orient.jpg Imageorient Tabea Oertel, 2017-03-23 13:31
Actions #1

Updated by Frank Nägler about 7 years ago

  • Status changed from New to In Progress
  • Assignee set to Frank Nägler
Actions #2

Updated by Gerrit Code Review about 7 years ago

  • Status changed from In Progress to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52136

Actions #3

Updated by Frank Nägler about 7 years ago

@Tabea, I can't reproduce the problem but think I know what is wrong.
I have prepared a patch: https://review.typo3.org/#/c/52136/

please test if it fix the problem, I will provide the backport for 7.6 if you can confirm that the patch works.

Actions #4

Updated by Gerrit Code Review about 7 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52136

Actions #5

Updated by Tabea Oertel about 7 years ago

  • Assignee deleted (Frank Nägler)

I tested the Patch and it does fix the problem.
But we should check the existance of the element in javascript too before registering any eventhandler on it. So we need an $selectElement.length check in SelectSingleElement.js.

Actions #6

Updated by Frank Nägler about 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Gerrit Code Review about 7 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52143

Actions #8

Updated by Gerrit Code Review about 7 years ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52143

Actions #9

Updated by Frank Nägler about 7 years ago

  • Status changed from Under Review to Resolved
Actions #10

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF