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

Tab complete covers reply #5997

Open
turt2live opened this issue Jan 17, 2018 · 3 comments
Open

Tab complete covers reply #5997

turt2live opened this issue Jan 17, 2018 · 3 comments
Labels
A-Replies reply P1 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect X-Needs-Design

Comments

@turt2live
Copy link
Member

Description

image

Version information

  • Platform: web (in-browser)
  • Browser: Chrome 63
  • OS: Windows 10
  • URL: riot.im/develop
@lampholder
Copy link
Member

HMM. Not 100% obvious what to do. It's probably fine for the autocomplete to partially obscure the reply (like it partially obscures the timeline), but it does look glitchy that they occupy the exact same space.

It's probably harder to implement that it is to suggest, but could autocomplete be moved over ~10px to the right?

@lampholder lampholder added T-Defect ui/ux P1 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users A-Replies reply labels Jan 22, 2018
@t3chguy
Copy link
Member

t3chguy commented Feb 1, 2018

@lampholder trivial, legit a left: 10px in CSS.

image

whatcha think?

@turt2live
Copy link
Member Author

In combination with #5973 I think it looks good :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Replies reply P1 S-Minor Impairs non-critical functionality or suitable workarounds exist S-Tolerable Low/no impact on users T-Defect X-Needs-Design
Projects
None yet
Development

No branches or pull requests

4 participants