Bug #22784
Width of one paragraph of text in the Install Tool is too big
Status: | Closed | Start date: | 2010-06-01 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Steffen Gebert | % Done: | 0% |
|
Category: | - | Spent time: | - | |
Target version: | - | |||
TYPO3 Version: | 4.4 | Is Regression: | ||
PHP Version: | 5.2 | Sprint Focus: | ||
Complexity: |
Description
The text goes over the edge in the paragraph "[pageUnavailable_handling]"
Reproducable:
1. Login in to the Install Tool
2. Go to "All Configuration"
3. Jump to "[pageUnavailable_handling]"
(issue imported from #M14564)
Related issues
History
#1 Updated by Chris topher about 5 years ago
This is still an issue.
Interestingly it works correctly for (most?) other sections on that screen, but not for this one (although the sourcecode is basically the same...).
#2 Updated by Chris topher about 5 years ago
I just checked that: It works correctly for all(!) other sections. This is the only one with that problem.
#3 Updated by Andy Grunwald about 5 years ago
The Problem is the String "USER_FUNCTION:typo3conf/pageUnavailableHandling.php:user_pageUnavailable->pageUnavailable" whoch could not be breaked by the browser.
#4 Updated by Chris topher about 5 years ago
So a simple space behind the "->" would fix that. A nobrainer in the end...
#5 Updated by Chris topher about 5 years ago
Or even better: Adding the CSS definition
word-wrap: break-word;
can fix that.
#6 Updated by Marc Wöhlken over 4 years ago
@christopher:
Is word-wrap really part of the css standard? AFAIK it's an IE invention and obviously does not work under Firefox...
Just tried adding "word-wrap: break-word; white-space: -moz-pre-wrap;" to the li-elements which worked fine.
Would it be appropriate to use such mozilla specific settings?!
#7 Updated by Chris topher over 4 years ago
word-wrap only is part of CSS3 I think. So, it is not yet a standard, but I read that it already should work in Firefox.
But the moz... property you propose is no standard either, is it? ;) Doesn't it work without that?
#8 Updated by Marc Wöhlken over 4 years ago
"word-wrap: break-word" seems not to word under FF 3.6.x.
#9 Updated by Chris topher over 4 years ago
OK.
Just take the solution you like best!
The same problem is there at 2 other places in the same file:
- The line displaying PATH_thisScript in Basic Configuration and
- The same line in the part "Clean up"
Would be great if you could fix that the same way then!
#10 Updated by Marc Wöhlken over 4 years ago
Added a patch to solve this issue:
- added new class t3-install-description to "All configuration page"
- formatted that class: word-wrap: break-word; width: 48em;
@christoph: word-wrap works in FF when the element has a defined width.
#11 Updated by Marc Wöhlken over 4 years ago
The same problem is there at 2 other places in the same file:
- The line displaying PATH_thisScript in Basic Configuration and
Works fine in TYPO3 4.4 / 4.5 under FF & IE8. Any info on which browsers have to be used to reproduce the problem?
- The same line in the part "Clean up"
Could not see where this problem arises.
#12 Updated by Chris topher over 4 years ago
- The line displaying PATH_thisScript in Basic Configuration
Indeed! I just saw the screenshot
http://bugs.typo3.org/file_download.php?file_id=11892&type=bug
which is attached to #22975.
I just noticed: For me a path, which would be too long, is broken at a "-" sign in the path. And paths without such a "-" are broken at the slash next to the border of the div box. Great!
- The same line in the part "Clean up"
Yes, sorry for the inconvenience. Seems like this line was removed since I last checked that.
#13 Updated by Marc Wöhlken over 4 years ago
This works even when there is no "-" in the path. In my installation the word "pageUnavailable" gets split up across lines.
#14 Updated by Chris topher over 4 years ago
Your patch works and solves the problem!
Send it to Core List!
#15 Updated by Marc Wöhlken over 4 years ago
Patch is submitted to core list, give it a +1 if you like the solution.
#16 Updated by Steffen Gebert over 4 years ago
Committed to trunk rev. 9816.
Thanks for your contribution, Marc!
#17 Updated by Susanne Moog over 4 years ago
- Target version deleted (
4.5.0)