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

[Run] Buttons overlap long result titles #8967

Closed
htcfreek opened this issue Jan 6, 2021 · 7 comments
Closed

[Run] Buttons overlap long result titles #8967

htcfreek opened this issue Jan 6, 2021 · 7 comments
Labels
Area-User Interface things that regard UX for PowerToys Priority-3 Bug that is low priority Product-PowerToys Run Improved app launch PT Run (Win+R) Window

Comments

@htcfreek
Copy link
Collaborator

htcfreek commented Jan 6, 2021

In the image you can see that the buttons overlap the long result title. This is something we should fix. Maybe by trimming the end of the visible text and replace it with ....

image
Image originally posted by @enricogior in #8076 (comment)

Update 2021-01-10 (htcfreek)
For the new Registry plugin we have a working solution in it's code.


@enricogior
I hope it's okay for you that I have used your screenshot.

@ghost ghost added the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jan 6, 2021
@htcfreek htcfreek changed the title [PT Run] UX Layout on long result title [PT Run] Buttons overlap long result titles Jan 6, 2021
@enricogior enricogior added the Product-PowerToys Run Improved app launch PT Run (Win+R) Window label Jan 6, 2021
@crutkas crutkas removed the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jan 6, 2021
@crutkas
Copy link
Member

crutkas commented Jan 6, 2021

i honestly don't think this is an issue. there should be a tooltip with full name.

@crutkas
Copy link
Member

crutkas commented Jan 6, 2021

we've had the UI like this since we rolled out

@htcfreek
Copy link
Collaborator Author

htcfreek commented Jan 7, 2021

I personally think it looks not nice. Maybe we can add a hiding effect for text behind and around buttons. So we then have a min. space between button area and result title.

As I know the tooltip exists per default for every plugin. Correct? If not, I am going to open a tooltip issue for the service plugin.

@crutkas
Copy link
Member

crutkas commented Jan 7, 2021

Tooltip is done by default from the template. I think everything has pro's and con's. Even then, this string will overflow.

I can't see an easy fix for this and question if it is really an issue as it has been there, like i stated, since we released back in May 2020

@htcfreek
Copy link
Collaborator Author

htcfreek commented Jan 7, 2021

I can't see an easy fix for this and question if it is really an issue as it has been there, like i stated, since we released back in May 2020

I would classify it as layout enhancement.

@enricogior enricogior added the Priority-3 Bug that is low priority label Jan 9, 2021
@enricogior enricogior changed the title [PT Run] Buttons overlap long result titles [Run] Buttons overlap long result titles Jan 9, 2021
@niels9001 niels9001 added the Area-User Interface things that regard UX for PowerToys label Jun 5, 2021
@niels9001 niels9001 self-assigned this Jun 5, 2021
@niels9001 niels9001 removed their assignment Oct 21, 2021
@htcfreek
Copy link
Collaborator Author

@niels9001
Does #28538 fix this issue?

@davidegiacometti
Copy link
Collaborator

Hi @htcfreek
Came across this issue while searching for one related to tooltips.
It's fixed by the UI rework.

image

@davidegiacometti davidegiacometti added Priority-3 Bug that is low priority and removed Priority-3 Bug that is low priority labels Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-User Interface things that regard UX for PowerToys Priority-3 Bug that is low priority Product-PowerToys Run Improved app launch PT Run (Win+R) Window
Projects
None yet
Development

No branches or pull requests

5 participants