Skip to content

Changed the behaviour of the masking mechanism: When the selector is …#692

Open
habruening wants to merge 2 commits intoOpenBoard-org:devfrom
habruening:1.7-dev_NoMasking
Open

Changed the behaviour of the masking mechanism: When the selector is …#692
habruening wants to merge 2 commits intoOpenBoard-org:devfrom
habruening:1.7-dev_NoMasking

Conversation

@habruening
Copy link

I changed the behaviour of the masking mechanism. In the current version it does not work. If the user has added annotations over the desktop, he has no chance to work with the desktop any more. The masking mechanism prevents him from interacting with the desktop.

So I improved this behaviour. As the selector button has no use in that case (because you cannot click through annotations anyway), I changed the behaviour of the selector. It hides all annotations and minimizes the desktop palette. Then the user can operate with the desktop. When he wants the annotations back, he just maximizes the desktop palette again.

Hiding the desktop palette may be a bit surprising in this situation. But as the user cannot draw annotations, there is actually no real need of having the desktop palette maximized.

…clicked in desktop mode, all annotations disappear so that the user can work with the desktop.

Platforms where the masking mechanism is not needed (Windows) are not affected. The original behaviour in this case has not been changed.
@kaamui kaamui changed the base branch from 1.7-dev to dev October 6, 2023 09:15
@habruening
Copy link
Author

Pull request #895 is an alternative to this.
This pull request solves the problem by not showing the annotations in selector mode, at all. Most teachers are happy with this solution.
Pull request #895 solves the same problem by making the board transparent for mouse requests. This is the better solution, but uses unusual features of Qt, that may be removed in future Qt versions. In case request #895 is accepted, this request becomes obsolete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant