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

UI improvement: Autoplay function needs better icon and description #1539

Closed
LavRadis opened this issue Jun 1, 2018 · 7 comments
Closed
Labels
area: ux area: viewer css help wanted level: 2 Some knowledge of the existing code is recommended

Comments

@LavRadis
Copy link
Contributor

LavRadis commented Jun 1, 2018

The Issue

The new "Autoplay" function sticks out of the UI design and looks out of place. It needs a better icon and placment.
It also needs some sort of explanation. I was confused about what it exactly does, so it could confuse other users with less experience.
The explanation could be in the form of hover-over description and/or helper text in "Settings"

System Configuration

  • LBRY Daemon version: 0.19.3
  • LBRY App version: 0.22.0-rc.9
  • LBRY Installation ID: 7tYN144rHXrspzJG9xLP28fi1JztXnbpKro6DBJEQyzBmXTyDgqRLfYsUvHYFVmMe2
  • Operating system: Windows (Windows-10-10.0.16299)

Screenshots

autoplay

@neb-b
Copy link

neb-b commented Jun 3, 2018

I am thinking something like air bnb's map switch?
screen shot 2018-06-03 at 3 50 42 pm

Here is how youtube presents it:
screen shot 2018-06-08 at 12 48 10 am

@neb-b
Copy link

neb-b commented Jun 3, 2018

This will be a global input style to replace the checkbox. We can also add a tooltip for a larger description on this.

@tzarebczan tzarebczan added help wanted level: 2 Some knowledge of the existing code is recommended and removed needs: triage labels Jun 4, 2018
@neb-b neb-b mentioned this issue Jun 15, 2018
@tzarebczan
Copy link
Contributor

We added the toggle, now just need the tool tip.

@neb-b
Copy link

neb-b commented Jun 16, 2018 via email

@neb-b
Copy link

neb-b commented Jun 16, 2018 via email

@tzarebczan
Copy link
Contributor

Yep, that's what I tried first haha and didn't bother checking the label.

@tzarebczan
Copy link
Contributor

This is fixed in the redesign, still looking for the exact placement though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ux area: viewer css help wanted level: 2 Some knowledge of the existing code is recommended
Projects
None yet
Development

No branches or pull requests

5 participants