Automattic/jetpack

Add "adapt_container_width" to facebook likebox widget

Offen

#9.889 geöffnet am 10.07.2018

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (1 zugewiesene Person)PHP (877 Forks)user submission
EnhancementGood For CommunityHacktoberfest[Feature] Extra Sidebar Widgets[Pri] Low

Repository-Metriken

Stars
 (1.808 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4T 5h) (645 gemergte PRs in 30 T)

Beschreibung

Facebook allows you to lower the width of the widget and make it adaptive. Unfortunately, this can not be played using the jetpack widget

Steps to reproduce the issue

  1. Create facebook likebox widget
  2. Set width to "0"

What I expected

The widget adapts to the width of the container

<div class="fb-page" data-adapt-container-width="true">...</div>

What happened instead

The widget has a static minimum width

<div class="fb-page" width="180" data-adapt-container-width="false">...</div>

Contributor Guide

Research-Richtung
Untersuche den Code des Facebook Likebox Widgets in Jetpack und finde die Stelle, an der das fb page div gerendert wird. Füge das Attribut data adapt container width mit einem Wert hinzu, der auf einer neuen Option basiert (z. B. einer Checkbox im Widget Formular). Aktualisiere die Methode update() des Widgets, um die Option zu speichern und die Ausgabe entsprechend zu ändern.
Tech Stack
php
Domain
backendfrontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitPHPWordPress
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75