incubateur-ademe / territoires-en-transitions

Plateforme numérique pour faciliter et accélérer la mise en oeuvre des actions de transition écologique dans les collectivités territoriales
Other
18 stars 7 forks source link

Retours de tests et cleaning sur la nouvelle fiche action #3289

Closed mariheck closed 3 weeks ago

cparthur commented 3 weeks ago

Dans le compo <SideMenu /> il faudrait changer le contenu du return par:

    <div
      className={classNames(
        'absolute z-[800] top-0 h-screen max-h-screen overflow-hidden w-[50%] lg:w-[33%] xl:w-[30%] flex flex-col bg-white rounded-l-lg border-l border-l-grey-3 transition-all duration-500',
        {
          '-right-[50%] lg:-right-[33%] xl:-right-[30%]': !isOpen,
          'right-0': isOpen,
        }
      )}
      style={isOpen ? {boxShadow: '-5px 0px 10px 1px #ddd'} : {}}
    >
      {/* Titre + bouton de fermeture */}
      <div className="p-4 flex justify-between items-center bg-white border-b border-b-grey-3 shadow-sm z-[801] top-0">
        <h5 className="mb-0">{title}</h5>
        <Button
          title="Fermer"
          onClick={() => setIsOpen(false)}
          icon="close-line"
          variant="grey"
          size="xs"
        />
      </div>

      {/* Contenu du side menu */}
      <div className="grow flex flex-col overflow-hidden">
        <div className="grow flex flex-col overflow-y-auto">{displayContent && children}</div>
      </div>
    </div>

et changer <div className="grow p-4 overflow-y-auto"> par <div className="p-4"> dans <Content />.

Ça fix le problème d'avoir le scroll qui commence depuis le titre et ça rend plus simple à utiliser sans avoir à setup l'overflow-y dans son utilisation