Project

General

Profile

Actions

Bug #61656

closed

Icon size in the groupMods field not set to 16x16

Added by Eric Chavaillaz about 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-09-16
Due date:
% Done:

100%

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

Description

When using high resolution icon (png) in backend modules (32x32 px) and this module can be enable/disable in the be_groups/be_users records with fields groupMods/userMods, the icon is displayed bigger than desired (see attachment).

To avoid this behaviour, I propose to introduce width and height attributes in the function "getIconHtml" of the formEngine class.


Files

IconTooBig.png (41.9 KB) IconTooBig.png Eric Chavaillaz, 2014-09-16 20:44

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #61813: Hard coded image sizes in backend formsClosed2014-09-23

Actions
Actions #1

Updated by Gerrit Code Review about 10 years ago

  • Status changed from New 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 http://review.typo3.org/32812

Actions #2

Updated by Xavier Perseguers about 10 years ago

  • Description updated (diff)
Actions #3

Updated by Gerrit Code Review about 10 years ago

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

Actions #4

Updated by Gerrit Code Review about 10 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32909

Actions #5

Updated by Anonymous about 10 years ago

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

Updated by Oliver Hader about 10 years ago

Regression... see issue #61813

Actions #7

Updated by Eric Chavaillaz about 10 years ago

If we use the first patch and modify all the method "getIconHtml" according to the new signature like Xavier said, the regression will disappear.
I can prepare a patch this evening.

Actions #8

Updated by Oliver Hader about 10 years ago

  • Status changed from Resolved to Accepted

Changes for TYPO3_6-2 and master have been reverted.
http://forum.typo3.org/index.php/t/206216/

Actions #9

Updated by Eric Chavaillaz over 9 years ago

It seems to be resolved with CSS in the TYPO3 7 version...

Thanks
Eric

Actions #10

Updated by Riccardo De Contardi about 9 years ago

  • Status changed from Accepted to Closed

I close it for now: it has been fixed in v7 using css; It will not be fixed in 6.2 due to huge refactorings in v7, thus backporting is not straightforward.

If you think that this is the wrong decision, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.

Actions

Also available in: Atom PDF