TYPO3 Mailingliste: nicht fragen - lesen!

Hy;

Here is a little suggestion to improve usability of Templa Voila:

------------------------------------------------------------

<div id="templavoila-preview-5d6d4a40c6" style="text-align: left;
position: absolute; display: none;">

-- table --
</div>
<div id="templavoila-preview-5d6d4a40c6-wrapper"
onmouseout="

document.getElementById('templavoila-preview-5d6d4a40c6').style.display='none';

document.getElementById('templavoila-preview-5d6d4a40c6-wrapper').attributes.getNamedItem('style').nodeVal ue
= '';
"
onmouseover="

document.getElementById('templavoila-preview-5d6d4a40c6').style.display='block';

document.getElementById('templavoila-preview-5d6d4a40c6-wrapper').attributes.getNamedItem('style').nodeVal ue
= 'border: 2px dashed #333366;';
"
style="">

-- content --
</div>

------------------------------------------------------------

try this instead, please:

------------------------------------------------------------
..templavoila-preview-wrapper:hover {
background: red;
}
..templavoila-preview-wrapper .templavoila-preview-info {
display: none;
}
..templavoila-preview-wrapper:hover .templavoila-preview-info {
display: block;
}

<div id="templavoila-preview-5d6d4a40c6-wrapper"
class="templavoila-preview-wrapper">
<div id="templavoila-preview-5d6d4a40c6"
class="templavoila-preview-info">
-- table --
</div>

-- content --
</div>
------------------------------------------------------------

It solves several problems:

- no javascript needed, cleaner, faster, customizable without forced
"!important"
- hovering the table doesn't go mad (flicker) anymore
- the deactivation of 'border' doesn't reenforce calculation of
surrounding margins
anymore, so it stops the shifting down on hover

It leaves the content-layout basically unchanged while appearing (if you are
not actually using "tv-parent:first-child"). It comes without
complications, nothing
becomes worst, only better.

Ciao
Niels

_______________________________________________
TYPO3-project-templavoila mailing list
TYPO3-project-templavoila (AT) lists...netfielders.de
http://lists.netfielders.de/cgi-bin/...ct-templavoila

Hi Niels,

Could you please post a patch and open a bug report on bugs.typo3.org? I
think it would then be easier for all of us to test it.

Regards


--
Xavier Perseguers
http://xavier.perseguers.ch/en


Niels Fröhling wrote:
> Hy;
>
> Here is a little suggestion to improve usability of Templa Voila:
>
> ------------------------------------------------------------
>
> <div id="templavoila-preview-5d6d4a40c6" style="text-align: left;
> position: absolute; display: none;">
> -- table --
> </div>
> <div id="templavoila-preview-5d6d4a40c6-wrapper"
> onmouseout="
>
> document.getElementById('templavoila-preview-5d6d4a40c6').style.display='none';
>
>
> document.getElementById('templavoila-preview-5d6d4a40c6-wrapper').attributes.getNamedItem('style').nodeVal ue
> = '';
> "
> onmouseover="
>
> document.getElementById('templavoila-preview-5d6d4a40c6').style.display='block';
>
>
> document.getElementById('templavoila-preview-5d6d4a40c6-wrapper').attributes.getNamedItem('style').nodeVal ue
> = 'border: 2px dashed #333366;';
> "
> style="">
> -- content --
> </div>
>
> ------------------------------------------------------------
>
> try this instead, please:
>
> ------------------------------------------------------------
> .templavoila-preview-wrapper:hover {
> background: red;
> }
> .templavoila-preview-wrapper .templavoila-preview-info {
> display: none;
> }
> .templavoila-preview-wrapper:hover .templavoila-preview-info {
> display: block;
> }
> <div id="templavoila-preview-5d6d4a40c6-wrapper"
> class="templavoila-preview-wrapper">
> <div id="templavoila-preview-5d6d4a40c6"
> class="templavoila-preview-info">
> -- table --
> </div>
> -- content --
> </div>
> ------------------------------------------------------------
>
> It solves several problems:
>
> - no javascript needed, cleaner, faster, customizable without forced
> "!important"
> - hovering the table doesn't go mad (flicker) anymore
> - the deactivation of 'border' doesn't reenforce calculation of
> surrounding margins
> anymore, so it stops the shifting down on hover
>
> It leaves the content-layout basically unchanged while appearing (if you
> are
> not actually using "tv-parent:first-child"). It comes without
> complications, nothing
> becomes worst, only better.
>
> Ciao
> Niels
>
_______________________________________________
TYPO3-project-templavoila mailing list
TYPO3-project-templavoila (AT) lists...netfielders.de
http://lists.netfielders.de/cgi-bin/...ct-templavoila

Hy Xavier;
> Could you please post a patch and open a bug report on bugs.typo3.org? I
> think it would then be easier for all of us to test it.
Okay, I'm trying to do that, meanwhile maybe it's good to explain it
here anyway:

Well ... if you use the view-module or the admin-panels, the TV-modules
generates
additional markup to display a border around the TV-element (page or FCE) as
well as a table containing information about the used template.

In the way this is achieved, it creates some problems for the layout of
the page it
is embedded into, as well as having real implementation issues and
inconveniences.

I can make the patch, but you have to provide some prerequisites. You
have to have a
FCE-element in you verification-page in this way:

<div style="margin-bottom: 1em;">
this is some content with margin to the following FCE element
</div>
<div id="this_is_our_FCE_elements_body" style="margin-top: 1em;
margin-bottom: 1em;">
this is our FCE-element which has margin to above and below
</div>
<div style="margin-top: 1em;">
this is some content with margin to the preceding FCE element
</div>

Looks complicated, but is really basic HTML-stuff. You know that in CSS the
margins overlap each other, so effectively we just have 1em distance,
not 2em.
And now when TV sets the border to 2px something we receive 1em + 2px + 1em.
The introduction of the border forces the re-calculation of the layout
and shifts all
following content 2em down.
Doesn't look so good, and may even corrupt complicated layouts, and so on.
Okay, I stop here, it's just this little thing causes so much trouble,
it should be fixed.
I don't know the TV-code out of mind, but maybe I have to introduce
_CSS_DEFAULT
for it, if it doesn't exist. I hope that's allright.

See you
Niels


_______________________________________________
TYPO3-project-templavoila mailing list
TYPO3-project-templavoila (AT) lists...netfielders.de
http://lists.netfielders.de/cgi-bin/...ct-templavoila