Skip to content

Conversation

vincentvidal
Copy link

@vincentvidal vincentvidal commented Jan 30, 2020

When window_dialog_float is false, which is the case by default, floating dialog windows are enabled anyway. REGION_PLEASE_FLOAT should not be added to region flags in that case.

fixes #61

When window_dialog_float is false, which is the case by default, floating dialog windows are enabled anyway. REGION_PLEASE_FLOAT should not be added to region flags in that case.
@vincentvidal vincentvidal force-pushed the fix-window-dialog-float branch from 3e05649 to b8b0c41 Compare January 30, 2020 12:15
@raboof raboof requested a review from ideasman42 January 30, 2020 15:18
Copy link
Owner

@raboof raboof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See also #52 for context.

Seems valid to me

@knixeur
Copy link
Collaborator

knixeur commented Jan 30, 2020

If I understand correctly, now we should not get floating dialogs by default, which we should've been getting since this feature was implemented. Just a warning, I was never bothered by it.

@knixeur knixeur merged commit 66412de into raboof:master Jan 30, 2020
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.

New "display dialog as float" feature : problem with scratchpad and option to deactivate
3 participants