TYPO3 Core

Hi,
had some time to look at this during the weekend - this alignment business is tricky... I've prepared a patch, please have a look at the attached diff (hope this file is ok, it was the first time I made a diff file ;-)

As suggested in t.p.content-rendering there are three $tablecode variants according to the alignment, that is now done with margins for centering and float:right for right-alignment. Tested on IE 5&6, Firefox, Mozilla, NN 7, O7, Konqueror.

No further need for an allign-attribute. It's done with inline styles and should be allright for all doctypes, no need to combine it with the config.doctype setting.

Please test this out cause I'm not sure about the float solution. It needs a clearing div - might there be a situation where this intereferes with other floatin gblock-levels in the neigbourhood?

Greets, Torsten

The same problem occurs for the complete page alignment when using e.g. the GLueCK template. (I know it is old fashioned, but it does cover all my needs :-))

IE does not have a problem, while in firefox the alignment is always left (I guess that is the default).

I have tried to apply the patch provided by Torsten, but is complains about being a reverse patch ???
When applying with -R (for reverse) the code gets changed, but the result is the same: no allignment for the page.

D'oh,
looks like the <div style="text-align: center"> method is also used for the stdWrap.wrapAlign property... this is used in GLUECK for page alignment.

My patch takes only image alignment into account, so this is why it has'nt worked out for you Ron. For your GLUECK Template you can solve it by inserting the following into your Setup:

page.10.value = <table border="0" cellpadding="0" cellspacing="0" width="1" style="margin: 20px auto;">
page.stdWrap >
page.stdWrap.wrapAlign = center

Thanx for the quick reply Torsten. I had in the meantime hacked myself another way around it (in fact restoring the 3.7.0 source) by doing the following:
page.stdWrap {
offsetWrap = {$page.offset}
offsetWrap.stdWrap.wrapAlign >
offsetWrap.stdWrap.outerWrap = <div align='center'>|</div>
offsetWrap.tdParams = valign="top"
}

I suppose your way of working (with the auto) is cleaner, but the above works fine for firefox and IE, which are the only supported browsers on our intranet.

Workaround não funcionou . Postei o arquivo (class.tslib_content.rar) com as alteração nas linhas 1087 e 1098 . Apenas substitua o arquivo nos diretorios :
J:\tslib
J:\typo3\sysext\cms\tslib

Hi

The solution for the problem is between the lines 1087 and 1098. A small change gets the alignment running properly.
Copy the attached file into the directories below:
J:\tslib
J:\typo3\sysext\cms\tslib
Remember to back up the original file, better safe than sorry !

OBS: TYPO 3.8 VERSION

@Luis: I deleted your attachment and replaced it with bug_1115.diff, doing exactly the same, but much easier to read :-)

@all review: We are going to release TYPO3 3.8.1 soon, and this release should definitely fix this bug. Unfortunately I have no time to test Luis' patch nor have I time to check the 2nd fix proposal which I have attached in bug_1115_solution2.diff.

If anybody has time to finish this, that would be very great!

- michael

image_align_fix.diff contains the final solution as proposed by Martin Kutschker in the content rendering list.

Please give feedback ASAP since 3.8.1 is close to be ready for release.

Fixed in 3.8.1. Thanks Masi!!