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

[GUI] Multiple cursors in labels #8225

Closed
sampletext32 opened this issue May 14, 2020 · 3 comments · Fixed by #8260
Closed

[GUI] Multiple cursors in labels #8225

sampletext32 opened this issue May 14, 2020 · 3 comments · Fixed by #8260
Labels

Comments

@sampletext32
Copy link
Contributor

On the very latest master I suddenly found this behaviour. I can place multiple cursors in labels.
image

This one is always displayed when I open a Help window
image

I think, this is undesired.

My PC Specs:
Core I5 10210U (4c/8t) 2.1 Ghz
UHD630 / MX250 2Gb
8Gb DDR4 SO-DIMM 2667Mhz
1920*1080 100% Scale Monitor
Note: I don't test games with this rig.

@sampletext32
Copy link
Contributor Author

Can reproduce on another rig.
image
9750H
HD630 / RTX2060
1920*1080 100% Scale Display.

@sampletext32
Copy link
Contributor Author

@Megamouse, I've made some research and found out that about_dialog.ui contains the exact same header as welcome_dialog.ui. So I compared the markup and found the difference.

<property name="textInteractionFlags">
    <set>Qt::TextSelectableByKeyboard|Qt::TextSelectableByMouse</set>
</property>

It seems like this small piece of code makes it possible to select texts in about_dialog. But it also introduces this multiple cursors behaviour.
So there's a question. If it is really needed the texts to be copyable, than my Issue is useless. If it's not - than we can disable text copy, and multiple cursors disappear.
Waiting for your reply, so I can create a PR or close Issue.

@illusion0001
Copy link
Contributor

I thought the behavior of those dialog boxes are read only and text can't be selected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants