nextcloud/nextcloud.com

Remove all mobile Popups

Open

#264 opened on Nov 11, 2016

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)PHP (126 forks)auto 404
bugdesignenhancementhelp wanted

Repository metrics

Stars
 (215 stars)
PR merge metrics
 (PR metrics pending)

Description

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 direction
Investigate the current popup implementation in the codebase, design an overlay solution that covers the whole screen, and ensure it works well on mobile devices. Focus on making the close button large and placing selectors at the top for easy tab switching.
Tech stack
phpcssjavascript
Domain
frontenddesign
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
HTMLCSSJavaScript
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80