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

Showing dialogs externally doesn't work? Always shows inside window #3729

Closed
kennethVDL opened this issue Jan 25, 2020 · 7 comments
Closed

Comments

@kennethVDL
Copy link

Hey guys, I can't seem to get the ShowModalMessageExternal method from MetroWindow to actually show outside the application?

Even in the Demo (using the net47 version), when showing the different dialogs, they all show up inside the main window, EXCEPT for the customdialog; that one does show externally.

Screenshots:
image
Becomes:
image

However,
image
Becomes:
image

Is there something wrong with my configuration or is this a bug?

Thanks in advance!

Kind regards,
Kenneth

@kennethVDL
Copy link
Author

Hey everyone, it's been 10 days without any answer. It's the first time I commented on a GitHub page with an issue. If this is not how I should've flagged this to you, please show me what the usual path to take is.
Thanks.

@timunie
Copy link
Collaborator

timunie commented Feb 4, 2020

Hi @kennethVDL ,

sorry for the delay. I can confirm that there is a different behavior for CustomDialog. But I don't know which one is the intended one or where is the root cause.

To your second question: It is always good to tell what is the current version you use. See also here: https://github.com/MahApps/MahApps.Metro/blob/develop/.github/CONTRIBUTING.md#bugs

Please note: This is a free time project maintained by mainly one person (not me) at the moment. Due to this there may be a delay in the answer / fix. I really hope for your understanding

Happy coding, Tim

@punker76
Copy link
Member

punker76 commented Feb 4, 2020

@kennethVDL I'm sorry nobody answered directly. Yes, I'm the maintainer of this project, but I have also a full time job and a wonderful family...

Yes, the ShowModal<XYZ>External method works different to the ShowDialogExternally and ShowModalDialogExternally methods.

All of these creates a new Window for the Dialog and shows it up. Only the ShowModal<XYZ>External method uses always the actual width of the owner (the main window).

So maybe, we can change this sometimes. Feel free to create a PR and we can discuss about this.

@kennethVDL
Copy link
Author

Hello @timunie and @punker76 , thanks for your replies! I honestly didn't try to sound rude and if I did, then I'd like to apologize. I'm just new to GitHub and was wondering if I maybe misunderstood how it worked! I appreciate your project and it has helped me immensely!

If I'm not mistaken I'm using version 1.6.5.

In any case, I thought the way it would work is that an external Dialog would show outside of the application (so using the actual width of the main screen) would make sense to me. And then the normal (non-external) dialogs would show inside the window that calls it.

If that's not the intended design then I will have to look into making custom versions of those Dialogs.

Just to be sure, I will open a pull regarding my question, thank you! ( Should I close this after opening the PR?)

@kennethVDL
Copy link
Author

Apologies, it seems the pull request requires that I make the change to the code myself and then show you exactly what I changed by comparing my own branch to the main one.

I will have to look into this later :|

@timunie
Copy link
Collaborator

timunie commented Feb 4, 2020

@kennethVDL no in the PR you would reference this and close it there. Have a look at some existing PR to understand what I am talking about (e.g. #3527).

Happy coding
Tim

@punker76
Copy link
Member

punker76 commented Nov 7, 2020

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version. If it’s an enhancement and hasn’t been taken on for so long, then it seems no one has the time to implement this.

Please reopen if you still encounter this issue with the latest stable version and then please use the issue template.

If you have enough time then it would be nice to test this issue against the latest alpha version (availabe via NuGet and appveyor).

Thank you! 🙏

@punker76 punker76 closed this as completed Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants