search

Ruben2776 / PicView

Fast, free and customizable image viewer for Windows 10 and 11.
https://picview.org
Other
995 stars 66 forks source link

Fullscreen - Image Info Crash #48

Closed Yevrag35 closed 2 years ago

Yevrag35 commented 2 years ago

Describe the bug When in fullscreen, attempting to show the 'Image Info' window crashes the app.

To Reproduce Steps to reproduce the behavior:

  1. Open any image.
  2. Go into Fullscreen mode.
  3. Right-click the image, and select "Image Info".

Expected behavior The "Image Info" window (box) should present itself the same way the button works when not in fullscreen mode.

Additional context The issue looks to be a null reference exception from UILogic\ConfigureWindow.cs -- https://github.com/Ruben2776/PicView/blob/f0cdcfdf03db191c24c522d02e6b4ca51c3562a2/PicView/UILogic/ConfigureWindows.cs#L158

It can be fixed by changing the line to

GetImageInfoWindow.Topmost = true;
Ruben2776 commented 2 years ago

Hi there,

Thanks for bringing this issue to my attention and providing a fix for it. The issue has been fixed in the 1.5.1 release. Have a great day!