-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[15.0][IMP] web_dialog_size: hide restore button when opening dialog #2821
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens if the system parameter is set to maximize by default?
0018128
to
1ec0106
Compare
Hello @pedrobaeza @chienandalu , look like there is a bug. I set 'maximize by default' but it did not work. I just added a few lines to fix that. Please help to review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with both configs. Thanks! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge patch
Is this applicable to v16?
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 3a08e54. Thanks a lot for contributing to OCA. ❤️ |
Context