fix file dialog resize method, ensure popup overlays are refreshed wh…

…en windows is created and resized, fixes fyne-io#1692 when fyne-io#1931 is landed

dialog/file.go

@@ -476,21 +476,23 @@ func (f *FileDialog) Resize(size fyne.Size) {
 	if f.dialog == nil {
 		return
 	}
-	maxSize := f.dialog.win.Size()
 	minSize := f.dialog.win.MinSize()
-	newWidth := size.Width
-	if size.Width > maxSize.Width {
-		newWidth = maxSize.Width
-	} else if size.Width < minSize.Width {
-		newWidth = minSize.Width
-	}
-	newHeight := size.Height
-	if size.Height > maxSize.Height {
-		newHeight = maxSize.Height
-	} else if size.Height < minSize.Height {
-		newHeight = minSize.Height
-	}
-	f.dialog.win.Resize(fyne.NewSize(newWidth, newHeight))
+	f.dialog.win.Resize(minSize.Max(size))
+	// maxSize := f.dialog.win.Size()
+	// minSize := f.dialog.win.MinSize()
+	// newWidth := size.Width
+	// if size.Width > maxSize.Width {
+	// 	newWidth = maxSize.Width
+	// } else if size.Width < minSize.Width {
+	// 	newWidth = minSize.Width
+	// }
+	// newHeight := size.Height
+	// if size.Height > maxSize.Height {
+	// 	newHeight = maxSize.Height
+	// } else if size.Height < minSize.Height {
+	// 	newHeight = minSize.Height
+	// }
+	// f.dialog.win.Resize(fyne.NewSize(newWidth, newHeight))
 }
 
 // Hide hides the file dialog.

internal/driver/glfw/canvas.go

@@ -178,7 +178,8 @@ func (c *glCanvas) Resize(size fyne.Size) {
 		if p, ok := overlay.(*widget.PopUp); ok {
 			// TODO: remove this when #707 is being addressed.
 			// “Notifies” the PopUp of the canvas size change.
-			p.Resize(p.Content.Size().Add(fyne.NewSize(theme.Padding()*2, theme.Padding()*2)))
+			// fmt.Printf("%#v ,canvas resize: %v\n", p, p.Content.Size())
+			p.Refresh()
 		} else {
 			overlay.Resize(size)
 		}

internal/driver/glfw/window.go

@@ -402,6 +402,14 @@ func (w *window) doShow() {
 	if w.canvas.Content() != nil {
 		w.canvas.Content().Show()
 	}
+
+	// Refresh popup overlays
+	for _, overlay := range w.canvas.Overlays().List() {
+		if _, ok := overlay.(*widget.PopUp); !ok {
+			continue
+		}
+		overlay.Refresh()
+	}
 }
 
 func (w *window) Hide() {

widget/popup.go

@@ -47,8 +47,10 @@ func (p *PopUp) Move(pos fyne.Position) {
 //
 // Implements: fyne.Widget
 func (p *PopUp) Resize(size fyne.Size) {
+	canvasSize := p.Canvas.Size()
 	p.innerSize = size
-	p.BaseWidget.Resize(p.Canvas.Size())
+	p.Content.Resize(p.innerSize.Subtract(fyne.NewSize(theme.Padding()*2, theme.Padding()*2)))
+	p.BaseWidget.Resize(canvasSize)
 	// The canvas size might not have changed and therefore the Resize won't trigger a layout.
 	// Until we have a widget.Relayout() or similar, the renderer's refresh will do the re-layout.
 	p.Refresh()