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.

Sign up for GitHub

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

Jump to bottom

No padding between "Clear" and scrollbar in the new search dialog #22554

Closed
Tracked by #21121
kittykat opened this issue Jun 13, 2022 · 2 comments
Closed
Tracked by #21121

No padding between "Clear" and scrollbar in the new search dialog #22554

kittykat opened this issue Jun 13, 2022 · 2 comments
Assignees
@justjanne
Labels
A-New-Search-Experience The new search dialog available in Labs O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Z-Labs

Comments

@kittykat
Copy link
Contributor

Steps to reproduce

  1. Have new search experience enabled
  2. Perform a few searches until the scrollbar is added to the UI

Outcome

What did you expect?

Padding between the word "Clear" and scrollbar

What happened instead?

No padding between "Clear" and scrollbar

Screenshot from 2022-06-13 14-47-25

Operating system

No response

Browser information

No response

URL for webapp

develop.element.io

Application version

No response

Homeserver

No response

Will you send logs?

No
@kittykat kittykat added T-Defect S-Tolerable Low/no impact on users O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-New-Search-Experience The new search dialog available in Labs Team: Delight labels Jun 13, 2022
@kittykat kittykat mentioned this issue Jun 13, 2022
Closed
21 tasks
@niquewoodhouse
Copy link

niquewoodhouse commented Jun 13, 2022 •
edited
Loading

I don't seem to have this when testing the latest deployment on matrix-org/matrix-react-sdk#8707 - the scrollbar is on the edge of the dialog, so it doesn't overlap with clear.

image

@kittykat kittykat assigned justjanne and unassigned niquewoodhouse Jun 15, 2022
@kittykat
Copy link
Contributor Author

I can't reproduce on https://pr8707--matrix-react-sdk.netlify.app/ right now, so closing

@kittykat kittykat closed this as not planned Won't fix, can't repro, duplicate, stale Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@justjanne justjanne

Labels
A-New-Search-Experience The new search dialog available in Labs O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect Z-Labs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kittykat @niquewoodhouse @justjanne