[ 
https://issues.apache.org/jira/browse/DOXIASITETOOLS-172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15814484#comment-15814484
 ] 

Michael Benz commented on DOXIASITETOOLS-172:
---------------------------------------------

border="0" can be removed. I was overzealous in making sure no borders are 
displayed around the banner. That should be the users's choice.
If I use the same alt definition as in the fluido skin, unit test 
DefaultSiteRenderer fails:
{quote}
Message: Attribute "alt" is required and must be specified for element type 
"img".
{quote}
Therefore I need to leave the empty {code}alt=""{code} tag in case neither name 
nor alt are defined for the banner image.

> Width, height and border values not used for banner display
> -----------------------------------------------------------
>
>                 Key: DOXIASITETOOLS-172
>                 URL: https://issues.apache.org/jira/browse/DOXIASITETOOLS-172
>             Project: Maven Doxia Sitetools
>          Issue Type: Bug
>          Components: Site renderer
>    Affects Versions: 1.7.4
>            Reporter: Michael Benz
>            Priority: Minor
>         Attachments: enable-width-height-border-doxia-sitetools.patch
>
>
> Similar to the issue MSKINS-16 that was fixed for the Fluido skin the Default 
> Skin ignores the width and height and border settings in the site descriptor.
> The changes in site.xml were introduced here:
> DOXIASITETOOLS-58
> {code}
> <bannerLeft>
>       <name>Name</name>
>       <src>Logo.svg</src>
>       <width>30</width>
>       <height>10</height>
>       <title>Logo Title</title>
>       <border>2</border>
> </bannerLeft>
> {code}
> Created HTML:
> {code}
> <img src="Logo.svg" alt="Name" title="Logo Title">
> {code}
> The CSS in maven-base.css would anyways not allow for a border around any 
> image - one would need to create a new skin for borders or remove the 
> border:none from the img tag.
> {code}
> /* from maven-base.css */
> img {
>   border:none;
> }
> /* (...) */
> #banner img {
>   border: none;
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to