ampproject / ampproject/amp-react-prototype

Components and AMP layout rules

Ouverte
#35 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

TBD
Langage dominant
JavaScript
Étoiles
36
Forks
6
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently AMP elements are structured like this:

<amp-element>      <- AmpElement (custom element wrapper)
                     <- defines layout/size
  <container>      <- BaseElement (implementation)
                     <- adheres to the layout defined by AmpElement

Some of the most critical layout styles are:

  • display: block|etc (but not "inline");
  • proxies for width and height;
  • position: relative|absolute|etc (but not "static");
  • overflow: hidden.

React components follow AMP's BaseElement layer which means that they accept layout, but normally do not define their own static layout.

Besides layout, AmpElement is also responsible for showing placeholder and loading indicator. This is especially important since the placeholder and loading indicator are shown this way even before the BaseElement implementation script has been downloaded.

To still allow static layout in the React/AMP layer, we could provide AmpLayout component that could implement layout/placeholder/loading-indicator features via composition.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Examiner la structure existante des composants React/AMP et les responsabilités de BaseElement/AmpElement décrites dans l’issue. Déterminer si une couche de composition AmpLayout peut fournir une mise en page statique, des placeholders et des indicateurs de chargement tout en respectant les règles de mise en page AMP. La tâche est terminée lorsqu’une conception des composants et un périmètre d’implémentation sont convenus.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
frontend
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
30/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.