espace entre le header et le corps du forum sur téléphone

[Mandrell]

Détails techniques

Version du forum : phpBB2
Poste occupé : Administrateur
Navigateur(s) concerné(s) : Mozilla Firefox, Google Chrome
Capture d'écran du problème :

Voir l'image:

Personnes concernées par le problème : Tous les utilisateurs
Problème apparu depuis : depuis longtemps
Lien du forum : https://goodmorningny.forumactif.com/

Description du problème

bonjour ou bonsoir ^^

il y a un problème d'espace entre le header et le corps du forum sur le téléphone et cela sur tous les navigateurs. j'ai cru comprendre qu'il s'agissait de publicités de fa invisibles et qu'on pouvait s'en débarrasser grâce aux crédits. dans le doute, je voudrais vérifier s'il s'agit bien de cela ou s'il y a une quelconque erreur dans le codage du forum. je sais pas de quoi vous aurez besoin donc je reste à votre disposition
merci par avance ^^

[Toryudo]

Bonjour !

Après vérification, non, ce n'est pas une publicité qui provoque ça, c'est bien le code. Vous pourrez d'ailleurs facilement voir et comprendre (ou deviner) ce qui se passe si vous utilisez la version PC et que vous rétrécissez la fenêtre du navigateur : plus la fenêtre est petite, plus la bannière est petite... mais l'espace entre la bannière et le corps du forum, lui, reste le même.

C'est parce que dans le CSS, le logo (une image vide) possède un height: 430px
Ce sont ces 430px qui permettent un affichage correct sur PC, mais qui représentent définitivement trop d'espace pour une version téléphone, surtout quand la bannière, elle, devient si petite.

Est-ce que vous avez besoin d'aide pour une correction ou est-ce que vous essayez de voir de votre côté ?

[Mandrell]

coucou !
merci beaucoup pour votre réponse ^^
ah maintenant que vous le dites ça me semble logique en effet.
je n'ai pas codé le forum. quand j'ai essayé une correction ça a marché sur mon forum test mais rien ne change sur le forum communiqué dans mon premier post. du coup si vous avez une solution franchement je prends car là je sèche un peu

[Toryudo]

Alors je veux bien votre Template overall_header si possible.

Je pense que le principe, ça va être :
- de mettre la bannière (background) à la place du logo
- de modifier votre script qui affiche une bannière au hasard parmi deux, pour qu'il continue de fonctionner
- de retirer des <br> en trop, que je vois par-ci par-là

Est-ce que vous pourriez aussi faire une capture d'écran de votre rendu sur PC ? Comme le rendu diffère selon les résolutions d'écrans, je voudrais savoir sur quel résultat me baser pour que l'affichage après modifications soit aussi propre qu'avant.

Pour finir, une petite remarque : attention, vous n'avez pas le droit de masquer l'élément #fa_left comme vous le faites, c'est un copyright Forumactif obligatoire quand vous utilisez la toolbar. Vous ne pouvez le masquer qu'au moyen d'un package Premium.
Un petit coup de pub à Forumactif, qui nous aide à vivre et partager nos passions

[Mandrell]

bonjour !

oui voici le template concerné



oui j'ai essayé d'enlever des br qui se baladent sans fermeture, j'ignore s'il y en a d'autres dans les autres templates.
j'ai fait une capture d'écran, là voici ; https://i.ibb.co/cQVHmGx/Sans-titre-1.jpg
j'ai pas vu pour le copyright enlevé vu que je n'ai pas codé le forum mais je vais le remettre bien évidemment, je suis désolée pour ça ><
merci encore pour votre aide =)

[Toryudo]

Bonjour !

D'accord, il y a quelque chose que je n'avais pas saisi... vous avez mis votre script de changement de bannière dans le champ "description du forum", c'est original :p

Pour un meilleur référencement de votre forum, mieux vaut mettre une vraie description ici, quitte à la masquer sur la page avec du CSS. Même chose pour le titre du forum d'ailleurs, ce sera plus propre pour les moteurs de recherche.

Alors, concernant les modifications, j'ai justement inclus le script qui affiche une bannière aléatoirement directement dans le Template. Contrairement à ce que je disais dans mon post précédent, j'ai laissé le système de bannière intact, mais dans le script, j'ai ajouté un calcul de la hauteur du logo (invisible) en fonction de la résolution de l'écran. Si j'ai plutôt procédé comme ça, c'est pour éviter d'avoir le petit "sursaut" quand la page se charge. C'est plus esthétique !

Code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
      "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" dir="{S_CONTENT_DIRECTION}" id="min-width" lang="{L_LANG_HTML}"
    xml:lang="{L_LANG_HTML}" {NAMESPACE_FB_LIKE} {NAMESPACE_FB} {NAMESPACE_BBCODE}>
<head>
  <link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com/" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Abril+Fatface&family=Economica&display=swap" rel="stylesheet">
<link href="https://fonts.googleapis.com/css2?family=Lexend+Peta:wght@200&display=swap" rel="stylesheet">
<link href="https://fonts.googleapis.com/css2?family=Fira+Code:wght@500&display=swap" rel="stylesheet">

  <title>{SITENAME_TITLE}{PAGE_TITLE}</title>
  <meta http-equiv="content-type" content="text/html; charset={S_CONTENT_ENCODING}"/>
  <meta http-equiv="content-script-type" content="text/javascript"/>
  <meta http-equiv="content-style-type" content="text/css"/>
  <!-- BEGIN switch_compat_meta -->
  <meta http-equiv="X-UA-Compatible" content="IE={switch_compat_meta.VERSION}"/>
  <!-- END switch_compat_meta -->
  <!-- BEGIN switch_canonical_url -->
  <link rel="canonical" href="{switch_canonical_url.CANONICAL_URL}"/>
  <!-- END switch_canonical_url -->
  {META_FAVICO}
  {META}
  {META_FB_LIKE}
  <meta name="title" content="{SITENAME_TITLE}{PAGE_TITLE}"/>
  {T_HEAD_STYLESHEET}
  {CSS}
  <link rel="search" type="application/opensearchdescription+xml" href="/improvedsearch.xml" title="{SITENAME}"/>
  <link rel="search" type="application/opensearchdescription+xml"
        href="{URL_BOARD_DIRECTORY}/search/improvedsearch.xml" title="{SEARCH_FORUMS}"/>
  <script src="{JQUERY_PATH}" type="text/javascript"></script>
  <!-- BEGIN switch_recent_jquery -->
  <script src="{JS_DIR}jquery1.9/jquery-migrate-1.4.1.js" type="text/javascript"></script>
  <script src="{JQUERY_DIR}browser/v1.0/jquery.browser.min.js" type="text/javascript"></script>
  <script src="{JQUERY_DIR}support/jquery.support.js" type="text/javascript"></script>
  <!-- END switch_recent_jquery -->
  <script src="{JS_DIR}{L_LANG}.js" type="text/javascript"></script>
  {RICH_SNIPPET_GOOGLE}

  <!-- BEGIN switch_fb_login -->
  <script src="https://connect.facebook.net/{switch_fb_login.FB_LOCAL}/all.js" type="text/javascript"></script>
  <script src="{switch_fb_login.JS_FB_LOGIN}" type="text/javascript"></script>
  <!-- END switch_fb_login -->

  <!-- BEGIN switch_ticker -->
  <link type="text/css" rel="stylesheet" href="{JQUERY_DIR}ticker/ticker.css"/>
  <script src="{JQUERY_DIR}/ticker/ticker.js" type="text/javascript"></script>
  <!-- END switch_ticker -->

  <!-- BEGIN switch_ticker_new -->
  <script src="{JQUERY_DIR}jcarousel/jquery.jcarousel.js" type="text/javascript"></script>
  <script type="text/javascript">//<![CDATA[
      /* Definir le sens de direction en fonction du panneau admin */
      var tickerDirParam = "{switch_ticker.DIRECTION}";
      var slid_vert = false;
      var auto_dir = 'next';
      var h_perso = parseInt({switch_ticker.HEIGHT});

      switch (tickerDirParam) {
        case 'top' :
            slid_vert = true;
            break;
        case 'left':
            break;
        case 'bottom':
            slid_vert = true;
            auto_dir = 'prev';
            break;
        case 'right':
            auto_dir = 'prev';
            break;
        default:
            slid_vert = true;
      }

      $(document).ready(function () {
        var w_cont = $('#fa_ticker_container').width();

        if (w_cont > 0) {
            $('#fa_ticker_container').width(w_cont);

            /* Affichage de la liste */
            $('#fa_ticker_content').css('display', 'block');

            /* Calcul des dimensions du conteneur et des elements */
            var width_max = $('ul#fa_ticker_content').width();
            var width_item = Math.floor(width_max / {switch_ticker.SIZE});
            var height_max = h_perso;

            /* Calcul de la hauteur maximale du conteneur en fonction des elements et de la hauteur personnalisee dans l'admin */
            $('ul#fa_ticker_content li').each(function () {
              if ($(this).height() > height_max) {
                  height_max = $(this).height();
              }
            });

            /* Redimensionnement des elements et des images trop larges */
            $('ul#fa_ticker_content li').width(width_item).height(height_max).find('img').each(function () {
              if ($(this).width() > width_item) {
                  var ratio = $(this).width() / width_item;
                  var new_height = Math.round($(this).height() / ratio);
                  $(this).height(new_height).width(width_item);
              }
            });

            /* Redimensionnement et centrage du conteneur en mode vertical */
            if (slid_vert) {
              $('ul#fa_ticker_content').width(width_item).height(height_max).css('marginLeft', 'auto').css('marginRight', 'auto');
            }

            /* Initialisation du caroussel */
            $('#fa_ticker_content').jcarousel({
              vertical: slid_vert,
              wrap: 'circular',
              auto: {switch_ticker.STOP_TIME},
              auto_direction: auto_dir,
              scroll: 1,
              size: {switch_ticker.SIZE},
              height_max: height_max,
              animation: {switch_ticker.SPEED}
            });
        } else {
            $('ul#fa_ticker_content li:not(:first)').css('display', 'none');
            $('ul#fa_ticker_content li:first').css('list-style', 'none').css('text-align', 'center');
        }
      });
      //]]>
  </script>
  <!-- END switch_ticker_new -->

  <script type="text/javascript">//<![CDATA[
      $(document).ready(function () {
        <!-- BEGIN switch_enable_pm_popup -->
        pm = window.open('{U_PRIVATEMSGS_POPUP}', '_faprivmsg', 'HEIGHT=225,resizable=yes,WIDTH=400');
        if (pm != null) {
            pm.focus();
        }
        <!-- END switch_enable_pm_popup -->
        <!-- BEGIN switch_report_popup -->
        report = window.open('{switch_report_popup.U_REPORT_POPUP}', '_phpbbreport', 'HEIGHT={switch_report_popup.S_HEIGHT},resizable=yes,scrollbars=no,WIDTH={switch_report_popup.S_WIDTH}');
        if (report != null) {
            report.focus();
        }
        <!-- END switch_report_popup -->
        <!-- BEGIN switch_ticker -->
        $(document).ready(function () {
            Ticker.start({
              height: {switch_ticker.HEIGHT},
              spacing: {switch_ticker.SPACING},
              speed: {switch_ticker.SPEED},
              direction: '{switch_ticker.DIRECTION}',
              pause: {switch_ticker.STOP_TIME}
            });
        });
        <!-- END switch_ticker -->
      });

      <!-- BEGIN switch_login_popup -->
      var logInPopUpLeft, logInPopUpTop, logInPopUpWidth = {LOGIN_POPUP_WIDTH},
        logInPopUpHeight = {LOGIN_POPUP_HEIGHT}, logInBackgroundResize = true, logInBackgroundClass = false;
      <!-- END switch_login_popup -->

      <!-- BEGIN switch_login_popup -->
      $(document).ready(function () {
        $(window).resize(function () {
            var windowWidth = document.documentElement.clientWidth;
            var popupWidth = $("#login_popup").width();
            var mypopup = $("#login_popup");

            $("#login_popup").css({
              "left": windowWidth / 2 - popupWidth / 2
            });
        });
      });
      <!-- END switch_login_popup -->
      //]]>
  </script>
  {GREETING_POPUP}
  <!-- BEGIN switch_ticker_new -->
  <style>
      .jcarousel-skin-tango .jcarousel-item {
        text-align: center;
        width: 10px;
      }

      .jcarousel-skin-tango .jcarousel-item-horizontal {
        margin-right: {switch_ticker.SPACING}px;
      }

      .jcarousel-skin-tango .jcarousel-item-vertical {
        margin-bottom: {switch_ticker.SPACING}px;
      }
  </style>
  <!-- END switch_ticker_new -->
  {HOSTING_JS}

  <!-- BEGIN google_analytics_code -->
  <script type="text/javascript">
      //<![CDATA[
      (function (i, s, o, g, r, a, m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
      (i[r].q=i[r].q||[]).push(arguments)}, i[r].l = 1 * new Date();
      a = s.createElement(o),
        m = s.getElementsByTagName(o)[0];
      a.async = 1;
      a.src = g;
      m.parentNode.insertBefore(a, m)
      })
      (window, document, 'script', '//www.google-analytics.com/analytics.js', 'ga');
      ga('create', '{G_ANALYTICS_ID}', 'auto');
      ga('send', 'pageview');
      ga('set', 'anonymizeIp', true);

      <!-- BEGIN google_analytics_code_bis -->
      ga('create', '{G_ANALYTICS_ID_BIS}', 'auto', 'bis');
      ga('bis.send', 'pageview');
      ga('bis.set', 'anonymizeIp', true);
      <!-- END google_analytics_code_bis -->
      //]]>

  </script>
  <!-- END google_analytics_code -->

</head>
<body background="{T_BODY_BACKGROUND}" bgcolor="{T_BODY_BGCOLOR}" text="{T_BODY_TEXT}" link="{T_BODY_LINK}"
    vlink="{T_BODY_VLINK}">
<!-- BEGIN hitskin_preview -->
<div id="hitskin_preview" style="display: block;">
  <h1><img src="https://2img.net/i/fa/hitskin/hitskin_logo.png" alt=""/> <em>Hit</em>skin.com</h1>
  <div class="content">
      <p>
        {hitskin_preview.L_THEME_SITE_PREVIEW}
        <br/>
        <span>{hitskin_preview.U_INSTALL_THEME}<a
                  href="https://{hitskin_preview.U_RETURN_THEME}">{hitskin_preview.L_RETURN_THEME}</a></span>
      </p>
  </div>
</div>
<!-- END hitskin_preview -->

<!-- BEGIN switch_login_popup -->
<div id="login_popup" style="z-index: 10000 !important;">
  <table class="forumline" width="{LOGIN_POPUP_WIDTH}" height="{LOGIN_POPUP_HEIGHT}" border="0" cellspacing="1"
        cellpadding="0">
      <tr height="25">
        <td class="catLeft">
            <span class="genmed module-title">{SITENAME}</span>
        </td>
      </tr>
      <tr height="{LOGIN_POPUP_MSG_HEIGHT}">
        <td class="row1" align="left" valign="top">
            <div id="login_popup_buttons">
              <form action="{S_LOGIN_ACTION}" method="get">
                  <input type="submit" class="mainoption" value="{L_LOGIN}"/>
                  <input type="button" class="mainoption" value="{L_REGISTER}"
                        onclick="parent.location='{U_REGISTER}';"/>
                  <input id="login_popup_close" type="button" class="button" value="{L_DONT_DISPLAY_AGAIN}"/>
              </form>
            </div>
            <span class="genmed">{LOGIN_POPUP_MSG}</span>
        </td>
      </tr>
  </table>
</div>
<!-- END switch_login_popup -->
           
            <!-- BEGIN switch_like_popup -->
<div id="like_popup" style="z-index: 10000 !important;">
  <table class="forumline" width="{LIKE_POPUP_WIDTH}" height="{LIKE_POPUP_HEIGHT}" border="0" cellspacing="1"
        cellpadding="0">
      <tr height="25">
        <td class="catLeft">
            <span class="genmed module-title">{SITENAME}</span>
        </td>
      </tr>
      <tr height="{LIKE_POPUP_MSG_HEIGHT}">
        <td class="row1" align="left" valign="top">
            <div id="like_popup_buttons">
              <form action="{S_LOGIN_ACTION}" method="get">
                  <input type="submit" class="mainoption" value="{L_LOGIN}"/>
                  <input type="button" class="mainoption" value="{L_REGISTER}"
                        onclick="parent.location='{U_REGISTER}';"/>
                  <input id="like_popup_close" type="button" class="button" value="{L_LIKE_CLOSE}"/>
              </form>
            </div>
            <span class="genmed">{LIKE_POPUP_MSG}</span>
        </td>
      </tr>
  </table>
</div>
<!-- END switch_like_popup -->

<a name="top"></a>
{JAVASCRIPT}
           
            <table width="100%" cellspacing="0" cellpadding="0" border="0">
            <tr>
              <!-- BEGIN switch_logo_left -->
              <td><a href="{U_INDEX}"><img loading="lazy" src="{LOGO}" id="i_logo" border="0" alt="{L_INDEX}" vspace="1"/></a>
              </td>
              <!-- END switch_logo_left -->
              <td align="center" width="100%" valign="middle">
                  <!-- BEGIN switch_logo_center -->
                  <a href="{U_INDEX}"><img loading="lazy" src="{LOGO}" id="i_logo" border="0" vspace="1"/></a>
                  <br/>
                  <!-- END switch_logo_center -->
                  <div class="maintitle">{MAIN_SITENAME}</div>
                  <br/>
                  <span class="gen">{SITE_DESCRIPTION}<br/>&nbsp; </span>
              </td>
              <!-- BEGIN switch_logo_right -->
              <td><a href="{U_INDEX}"><img loading="lazy" src="{LOGO}" id="i_logo" border="0" alt="{L_INDEX}" vspace="1"/></a>
              </td>
              <!-- END switch_logo_right -->
            </tr>
        </table>

<script type="text/javascript">
  var banListe = ['1','2'];
  document.body.classList.add('ban' + banListe[Math.floor(Math.random() * banListe.length)]);

  var logoid = document.getElementById('i_logo');
  var height = logoid.offsetWidth * 760 / 1800;
  logoid.height = height;
</script>

<table class="bodylinewidth" width="{T_BODY_TABLE_WIDTH}" cellspacing="0" cellpadding="10" border="0" align="center">
  <tr>
     
        <td class="bodyline">
                         


        <table cellspacing="0" cellpadding="0" border="0" align="{MENU_POSITION}" class="navbar">
            <tr>
                                <td align="{MENU_POSITION}"{MENU_NOWRAP}>{GENERATED_NAV_BAR}</td>
            </tr>
        </table>

        <div style="clear: both;"></div>

        <!-- BEGIN switch_ticker_new -->
        <div id="fa_ticker_block" style="margin-top:4px;margin-bottom:4px;">
            <table width="100%" border="0" cellspacing="0" cellpadding="0" class="forumline">
              <tr>
                  <td align="left" class="row1">
                    <div id="fa_ticker_container">
                        <ul id="fa_ticker_content" class="jcarousel-skin-tango" style="display:none;">
                          <!-- BEGIN ticker_row -->
                          <li>{switch_ticker.ticker_row.ELEMENT}</li>
                          <!-- END ticker_row -->
                        </ul>
                    </div>
                  </td>
              </tr>
            </table>
        </div>
        <!-- END switch_ticker_new -->

        <!-- BEGIN switch_ticker -->
        <div id="fa_ticker_block" style="margin-top:4px;margin-bottom:4px;">
            <table width="100%" border="0" cellspacing="0" cellpadding="0" class="forumline">
              <tr>
                  <td align="left" class="row1">
                    <div id="fa_ticker_container">
                        <div id="fa_ticker" style="height:{switch_ticker.HEIGHT}px;">
                          <div class="fa_ticker_content">
                              <!-- BEGIN ticker_row -->
                              <div>{switch_ticker.ticker_row.ELEMENT}</div>
                              <!-- END ticker_row -->
                          </div>
                        </div>
                    </div>
                  </td>
              </tr>
            </table>
        </div>
        <!-- END switch_ticker -->

        <div id="page-body">
            <div id="{ID_CONTENT_CONTAINER}"{CLASS_CONTENT_CONTAINER}>
              <table cellpadding="0" cellspacing="0" width="100%" class="three-col">
                  <tbody>
                  <tr>
                    <td valign="top" width="{C1SIZE}">
                        <div id="{ID_LEFT}">
                          <!-- BEGIN giefmod_index1 -->
                          {giefmod_index1.MODVAR}
                          <!-- BEGIN saut -->
                          <div style="height:{SPACE_ROW}px"></div>
                          <!-- END saut -->
                          <!-- END giefmod_index1 -->
                        </div>
                    </td>
                    <td valign="top" width="100%">
                        <!-- BEGIN html_validation -->
                    </td>
                  </tr>
                  </tbody>
              </table>
            </div>
        </div>
      </td>
  </tr>
</table>
</body>
</html>
<!-- END html_validation -->

Il reste encore une chose à modifier dans votre CSS personnalisé, c'est l'élément #i_logo.
Pour l'instant, vous avez ceci :

Code:

#i_logo{
    height:430px;
    width:100%;
    top:0;
    margin-bottom:130px
}

Et il faut le changer en ceci :

Code:

#i_logo {
    width: 100%;
}

Après ces modifications, il reste encore un espace entre la bannière et le texte.
Il correspond à ces éléments dans le Template :

Code:

<div class="maintitle">{MAIN_SITENAME}</div>
<br/>
<span class="gen">{SITE_DESCRIPTION}<br/>&nbsp; </span>

- le sitname, titre du site (texte vide pour l'instant)
- un espace <br/>
- la description du site (votre script que vous devez effacer maintenant, parce qu'il est déplacé dans le Template)

Comme je le disais au début, je vous conseille de les remplir pour le référencement. Par contre, vous pouvez les masquer via du CSS, et/ou supprimer le <br/> du Template pour gagner encore un peu de place. Normalement, après tout ça, tout devrait être propre sur toutes les résolutions !

[Mandrell]

bonjour !
ah oui vous avez parfaitement raison pour les bannières aléatoires, c'est mieux d'inclure le script dans le template c'est plus propre ^^
j'ai fait les modifications et en effet c'est beaucoup mieux ! le rendu est pareil que ce soit l'ordinateur comme sur téléphone, merci infiniment
le problème est résolu du coup et merci encore une fois