Project

General

Profile

Actions

Bug #30900

closed

ActionIcon for Available Extensions is hidden in ExtensionManager

Added by Stefan Sorgen over 12 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Category:
Extension Manager
Target version:
Start date:
2011-10-13
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

In the backend tool "Extension Manager" the first tab show a list all available extensions. In the first column there should be an icon for the action "update", "install" or "remove".

Today I noticed, that there are only three dots "..." and the icons had gone for all extensions (each row) in this tab (but not for the tab import extensions).

I found out that the icon is still available, but there are hidden...

I use Firefox-7.0.1 and retreive yesterday an update. It could be possible, that with the update of FF the CSS compatibilities was extended.

My analysis:
- there are two img elements inside the td element, first for update, second for install/remove
- the css style for img is inherited from td which has the classes "paddingNoActionIcon" and "paddingActionIcon":
.paddingActionIcon img {margin-right:5px}
.paddingNoActionIcon img {margin-right:20px}
(declared in /typo3/sysext/em/res/css/t3_em.css)
- the css style for td has the class "x-grid3-cell-inner":
.x-grid3-cell-inner {
overflow: hidden;
padding: 3px 3px 3px 5px;
text-overflow: ellipsis;
white-space: nowrap;
}

Problem:
The CSS "text-overflow: ellipsis" means, that an overflow of content will not cut off, but replaced by a string "...".
The browser calculate the width and however include the margin to the width and notes that the image can not be displayed completely and replaces the second image with "...".

This property is not W3C conform and exists since IE5.5, but i don't know why it doesn't occur in IE8 and IE7.

Solutions:
When I disable the property "text-overflow: ellipsis" or change the "margin-right" of the definition ".paddingNoActionIcon img" to "5px", the icon will be displayed.
A change of the padding from "3px 3px 3px 5px" to "3px" works as well.

I hope you have an opportunity to check this.

I tested it in IE8 and FF6.0.2 and there it looks ok.


Files

screenshot_em_actionIcons.png (9.43 KB) screenshot_em_actionIcons.png Stefan Sorgen, 2011-10-13 17:27

Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #33931: sprites in em borken with ff 12/13 install/uninstallClosedFrancois Suter2012-02-14

Actions
Has duplicate TYPO3 Core - Bug #36665: Missing install/uninstall icons in Extension Manager of 4.6 and 4.7 FF12 and IE9Closed2012-04-27

Actions
Has duplicate TYPO3 Core - Bug #36770: Extension manager doesn't show whether extension is installedClosed2012-05-02

Actions
Actions #1

Updated by Stefan Sorgen over 12 years ago

Stefan Sorgen wrote:

- the css style for td has the class "x-grid3-cell-inner":
.x-grid3-cell-inner {

...

}

(declared in /typo3/contrib/extjs/resources/css/ext-all-notheme.css)

Actions #2

Updated by Stefan Sorgen over 12 years ago

The problem could be the language selector... which come with another bug (I think)

When you select a language in the 3th tab "Language Packages" (for example "de"), an entry will be added to "localconf.php"
After that, you will see in the first tab "Available Extensions" an arrow at each row (see screenshot) - this is the ExtJS function "row-expander"

The other bug I noted is, normally if you click on an extension, you will change the view to the extension details.
If you selected a language, you can't switch to this view - a double click on the row or a click to the arrow will expand a row below which is shows the extension details.
On this way, some input fields (used for extension configuration) are broken or don't work correct.

When I de-select any languages, the effect is still present.
In the "localconf.php" is still an entry:
$TYPO3_CONF_VARS['EXT']['extConf']['em'] = 'a:1:{s:17:"selectedLanguages";s:0:"";}';

Now I deleted the line manually and all works fine - there is now also more space (without the row-expander-arrow) for to display the actionIcons.

btw. tested in Typo3 4.6.0 [beta3] and [rc1]

I hope this will help...

Actions #3

Updated by Chris topher over 12 years ago

  • Project changed from 1250 to TYPO3 Core
Actions #4

Updated by Chris topher over 12 years ago

  • Category set to Extension Manager
  • PHP Version set to 5.3
  • TYPO3 Version set to 4.6
Actions #5

Updated by Francois Suter almost 12 years ago

  • Status changed from New to Accepted
  • Assignee set to Francois Suter
  • Target version set to 4.7.1

I found this report while looking at the same effect but that appeared for me after upgrading to FF12. I have a solution but no time to finalize it tonight. Will submit a patch soon, though.

Actions #6

Updated by Francois Suter almost 12 years ago

  • Complexity set to easy
Actions #7

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10809

Actions #8

Updated by Francois Suter almost 12 years ago

  • Target version changed from 4.7.1 to 4.6.9
Actions #9

Updated by Francois Suter almost 12 years ago

Stefan, could you test the patch I submitted, please, to see if it also solve the issues you encountered?

Thanks in advance.

Actions #10

Updated by Francois Suter almost 12 years ago

  • Priority changed from Should have to Must have
  • Target version changed from 4.6.9 to 4.5.16
  • TYPO3 Version changed from 4.6 to 6.0
Actions #11

Updated by Gerrit Code Review almost 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10809

Actions #12

Updated by Francois Suter almost 12 years ago

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

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/10877

Actions #14

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/10878

Actions #15

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/10879

Actions #16

Updated by Francois Suter almost 12 years ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Dirk Wildt almost 12 years ago

Hy guys,
A change of the padding from "3px 3px 3px 5px" to "3px" doesn't fix the missing icons in my installation.

I fixed it with:

typo3_src-4.6.8/typo3/contrib/extjs/resources/css/ext-all-notheme.css

...
.x-grid3-cell-inner, .x-grid3-hd-inner{
overflow:hidden;
o-text-overflow: ellipsis;
/* 120502, dwildt, 1
/
/
text-overflow: ellipsis; /
/
120502, dwildt, 1+ */
text-overflow: hidden;
padding:3px 3px 3px 5px;
white-space: nowrap;
}

  • TYPO3 4.6.8
  • Ubuntu 12.04
  • Firefox 12.0

Kind regards
Dirk

Actions #18

Updated by Francois Suter almost 12 years ago

Hi Dirk,

Indeed and that's not how it has been solved. The patch which was eventually committed to the Git repository overrides the ellipsis setting, as you mention.

Actions #19

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF