[TYPO3-dev] new extension: vx_xajax_shoutbox

[Thomas Mammitzsch]

Elmar Hinz schrieb:
> 
> Hi Thomas,
> 
> I enjoy to see the first extension in TER build upon xaJax( apart from the
> xaJax Tutor). I also like your coding style, which comes close to my vision
> of modern coding I started to describe in:
> 
> <http://typo3.org/documentation/document-library/extension-manuals/lib/current/>
> (The second part about the controllers is not up to date)
> 
thank you, i was hoping you enjoy that, because its built upon your
extension :)

> I didn't install it yet, but I observe this points in the code:
> 
> <http://typo3.org/typo3temp/tx_terfe/t3xcontentcache/v/x/vx_xajax_shoutbox-0.0.1-pi1__static__setup__txt>
> 
> ** I guess it should be a USER_IN object:
> 
it was USER_INT in the first time. But i'm not sure why not to cache
this one. JavaScript is doing his job and it seemed to work out...
>
> **  _CSS_DEFAULT_STYLE =  ...
> 
> Inline CSS. I would put it into a file and include the file in the header
> like you do it with the generated javascript. Configurable path via TS setup.
>  <http://typo3.org/extensions/repository/search/vx_xajax_shoutbox/0.0.1/details/>
> 
Thats a good idea, i'll change that. I didn't think about that, because
i know _CSS_DEFAULT_STYLE as the only common practice known from other
extensions. But as Martin wrote in this thread, i also clear the default
styles via TS myself. Its just to give it a face from the start.

> TER doesn't show dependency from xajax. Either you need to add it in
> ext_emconf.php or TER has a little bug.

i know that i wrote that to ext_emconf.php.
I also get an table error shown in the extension manager/info. No idea
about that.
> 
regards, Thomas