nextcloud/nextcloud.com

Remove all mobile Popups

Offen

#264 geöffnet am 11.11.2016

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)PHP (126 Forks)auto 404
bugdesignenhancementhelp wanted

Repository-Metriken

Stars
 (215 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In mobile is not a good pratice to use popups for example if you are scrolling down in a popup you scroll down the page itself to.

wp_ss_20161111_0001

What I suggest it to transform that in overlay that cover the whole screen. Put the selectors on top for more easy change of tab, and remove some of the sub-paragraphs in the titles. The X close button should be big to. I can design a proper mockup, but I want to know if you guys agree with me.

wp_ss_20161111_0001_2

@nextcloud/designers @jospoortvliet

Contributor Guide

Research-Richtung
Untersuche die aktuelle Popup Implementierung im Code, entwerfe eine Overlay Lösung, die den gesamten Bildschirm abdeckt, und stelle sicher, dass sie auf mobilen Geräten gut funktioniert. Konzentriere dich darauf, den Schließbutton groß zu machen und die Selektoren oben zu platzieren, um einfaches Umschalten der Tabs zu ermöglichen.
Tech Stack
phpcssjavascript
Domain
frontenddesign
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
HTMLCSSJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80