Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inserter: optimize for mobile #859

Closed
jasmussen opened this issue May 22, 2017 · 2 comments · Fixed by #1089
Closed

Inserter: optimize for mobile #859

jasmussen opened this issue May 22, 2017 · 2 comments · Fixed by #1089
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface General Interface Parts of the UI which don't fall neatly under other labels. Good First Issue An issue that's suitable for someone looking to contribute for the first time Mobile Web Viewport sizes for mobile and tablet devices [Type] Task Issues or PRs that have been broken down into an individual action to take
Milestone

Comments

@jasmussen
Copy link
Contributor

jasmussen commented May 22, 2017

Current look:

screen shot 2017-05-22 at 14 03 26

Suggested improvements:

  • Bigger touch targets
  • Close button
  • Perhaps full screen/modal experience like the sidebar

Edit: There's also a sweet mockup/animatic here, which we should look at for this #44 (comment)

@jasmussen jasmussen added General Interface Parts of the UI which don't fall neatly under other labels. Mobile Web Viewport sizes for mobile and tablet devices labels May 22, 2017
@jasmussen jasmussen modified the milestone: Beta Jun 5, 2017
@mtias mtias changed the title Mobile: Inserter should be mobile optimized Inserter: optimize for mobile Jun 6, 2017
@mtias mtias added [Feature] Inserter The main way to insert blocks using the + button in the editing interface Design [Type] Task Issues or PRs that have been broken down into an individual action to take labels Jun 6, 2017
@jwold
Copy link

jwold commented Jun 7, 2017

I'm thinking a separate screen makes more sense in this instance. Some phones are larger and could handle a small popup well, but for many others it won't look good.

@jasmussen
Copy link
Contributor Author

I'm thinking a separate screen makes more sense in this instance. Some phones are larger and could handle a small popup well, but for many others it won't look good.

I agree. No reason not to make this a big modal on the small screens. The popout is already dismissed when you click the block inside, so should be enough to CSS this up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface General Interface Parts of the UI which don't fall neatly under other labels. Good First Issue An issue that's suitable for someone looking to contribute for the first time Mobile Web Viewport sizes for mobile and tablet devices [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants