Les modèles d'éléments de flux utilisateur nécessitent une mise à jour

UX
,
1

Bonjour,

J’ai remarqué sur ces plugins que les templates user-stream-item nécessitent une mise à jour (peut-être que d’autres plugins aussi)… pour s’adapter aux changements du cœur.

Plugin Discourse Follow
https://github.com/discourse/discourse-follow/blob/main/assets/javascripts/discourse/templates/components/follow-feed-post.hbs

Plugin Discourse Reactions
https://github.com/discourse/discourse-reactions/blob/main/assets/javascripts/discourse/components/discourse-reactions-reaction-post.hbs

Maintenant, cela ressemble à ceci :arrow_down_small:

Screenshot 2023-01-31 at 11.49.46
Screenshot 2023-01-31 at 11.49.461102×190 8.54 KB

Ceci est attendu :arrow_down_small:

Screenshot 2023-01-31 at 12.37.49
Screenshot 2023-01-31 at 12.37.491078×146 8.3 KB

Il y a eu une correction sur le template principal user-stream-item il y a quelques mois, qui a changé l’ordre des éléments et ajouté de nouvelles classes.

Merci :slightly_smiling_face:

3 « J'aime »
2

Si quelqu’un a besoin d’une solution rapide, j’ai apporté quelques modifications CSS pour placer ces éléments au bon endroit. :slightly_smiling_face:

Commun > CSS

.user-stream {
  .item,
  .user-stream-item {
    // l'ajout de l'exclusion n'a pas beaucoup d'importance
    // sauf qu'il n'ajoutera pas ces éléments au style de base actuel
    .info:not(.user-stream-item__header) {
      display: flex;
      align-items: flex-start;
      .stream-topic-details {
        order: 1;
        flex-grow: 1;
      }
      .expand-item,
      .collapse-item {
        order: 2;
      }
      .time {
        order: 3;
      }
    }
  }
}
1 « J'aime »
9

Merci pour le rapport @Don ! Il semble que les plugins de réactions/suivi aient initialement copié/collé la structure HTML de base, et qu’ils dépendaient donc de l’ancien CSS.

Je pense que nous pouvons rendre les choses plus claires en réutilisant le composant <UserStreamItem> de base dans les deux plugins :

Cela ajoutera un peu plus de personnalisation de base : DEV: Add `{{yield}}` to user-stream-item for plugins/themes by davidtaylorhq · Pull Request #20791 · discourse/discourse · GitHub

Correction pour les réactions : UX: Re-use UserStreamItem from core by davidtaylorhq · Pull Request #217 · discourse/discourse-reactions · GitHub

Correction pour le suivi : UX: Re-use UserStreamItem from core by davidtaylorhq · Pull Request #91 · discourse/discourse-follow · GitHub

4 « J'aime »
10

Ce sujet a été automatiquement fermé après 37 heures. Les nouvelles réponses ne sont plus autorisées.