better_display_on_small_screens #24

Merged

sinavir merged 0 commits from refs/pull/24/head into master

2022-07-26 00:46:43 +02:00

sinavir commented

2022-07-18 22:03:04 +02:00

(Migrated from git.rz.ens.wtf)

Fix partially issue https://git.rz.ens.wtf/Klub-RZ/metis/issues/

The version of Metis at commit 8b6a9bbf5a is momentarily available at https://cal.sinavir.fr

Fix partially issue https://git.rz.ens.wtf/Klub-RZ/metis/issues/ The version of Metis at commit https://git.rz.ens.wtf/Klub-RZ/metis/src/commit/8b6a9bbf5aad2aab2b379d638940da5e4e6afb6b is momentarily available at https://cal.sinavir.fr

raito commented

2022-07-25 19:25:11 +02:00

(Migrated from git.rz.ens.wtf)

Franchement, j'ai rien contre, tu en penses quoi @tomate ?

tomate commented

2022-07-25 20:38:34 +02:00

(Migrated from git.rz.ens.wtf)

J'ai deux commentaires sur l'esthétique :

En version mobile il manque de l'espacement dans les titres du coup, je propose de rajouter

:global(.fc-toolbar-chunk:not(:last-child)) {
  margin-bottom: 0.25em;
}

Avoir des boutons puis le titre puis de boutons je trouve ça un peu moche, un truc plus élégant serait :

  const headers = mobile
    ? {
        left: 'title',
        center: 'prev,today,next',
        right: 'dayGridMonth,timeGridWeek,timeGridDay,listWeek'
      }
    : {
        left: 'prev,next today',
        center: 'title',
        right: 'dayGridMonth,timeGridWeek,timeGridDay,listWeek'
      };
      
...

    headerToolbar: headers,

J'ai deux commentaires sur l'esthétique : - En version mobile il manque de l'espacement dans les titres du coup, je propose de rajouter ```css :global(.fc-toolbar-chunk:not(:last-child)) { margin-bottom: 0.25em; } ``` - Avoir des boutons puis le titre puis de boutons je trouve ça un peu moche, un truc plus élégant serait : ```javascript const headers = mobile ? { left: 'title', center: 'prev,today,next', right: 'dayGridMonth,timeGridWeek,timeGridDay,listWeek' } : { left: 'prev,next today', center: 'title', right: 'dayGridMonth,timeGridWeek,timeGridDay,listWeek' }; ... headerToolbar: headers, ```

sinavir commented

2022-07-26 00:07:32 +02:00

(Migrated from git.rz.ens.wtf)

Tu as raison, je pousse

sinavir commented

2022-07-26 00:08:24 +02:00

(Migrated from git.rz.ens.wtf)

Bon on perd tout de même la fluidité mais bon, c'est un moindre mal

No reviewers

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DGNum/metis#24

No description provided.