fixed "Focus on banner should frame just the banner itself and not th…

…e whole line with the banner"
CeON · Dec 9, 2024 · 5f61464 · 5f61464
1 parent 07f3a49
commit 5f61464
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/...c/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/dto/ImageWithLinkDto.java b/...c/main/java/edu/harvard/iq/dataverse/bannersandmessages/banners/dto/ImageWithLinkDto.java
@@ -23,6 +23,7 @@ public void setImage(StreamedContent image) {
     public String getLink() {
         return link;
     }
+
 
     public void setLink(String link) {
         this.link = link;

diff --git a/dataverse-webapp/src/main/webapp/text-messagesAndBanners.xhtml b/dataverse-webapp/src/main/webapp/text-messagesAndBanners.xhtml
@@ -19,18 +19,18 @@
             </ui:repeat>
 
             <ui:repeat value="#{banners}" var="banner">
-                <div class="p-col">
-                    <h:commandLink style="display: block;"
+                <div style="display: flex;align-content: center;">
+                  <h:commandLink style="text-align: center; display: block; margin: auto;  padding: 10px; "
                                    target="_blank" 
                                    disabled="#{banner.link == null}" 
                                    action="#{MessagesAndBannersFragment.redirect(banner.link)}">
-                         <p:graphicImage style="display: block ;margin-left: auto; margin-right: auto; padding: 2px;"
+                         <p:graphicImage 
                                         stream="false"
                                         value="#{banner.image}"
                                         alt="#{bundle['header.banner.alt']} #{banner.link != null ? bundle['opensInNewTab'] : ''}"
                                         title="#{bundle['header.banner.alt']} #{banner.link != null ? bundle['opensInNewTab'] : ''}"
                         />
-                    </h:commandLink>
+                 </h:commandLink> 
                 </div>
             </ui:repeat>
         </div>