Screen Module, window titlebar, position, size, DPI

CHANGELOG.md

@@ -9,10 +9,18 @@ You can find its changes [documented below](#060---2020-06-01).
 
 ### Added
 
+- Windows: Added Screen module to get information about monitors and the screen. ([#1037] by [@rhzk])
+- Added documentation to resizable() and show_titlebar() in WindowDesc. ([#1037] by [@rhzk])
+- Windows: Added internal functions to handle Re-entrancy. ([#1037] by [@rhzk])
+- Windows: WindowDesc: Create window with disabled titlebar, maximized or minimized and with position. ([#1037] by [@rhzk])
+- Windows: WindowHandle: Toggle maximize, minimize window. Toggle titlebar. Change size and position of window. ([#1037] by [@rhzk])
+- Windows: WindowHandle: Added handle_titlebar(), Allowing a custom titlebar to behave like the OS one. ([#1037] by [@rhzk])
 - Added ctrl/shift key support to textbox. ([#1063] by [@vkahl])
 
 ### Changed
 
+- Windows: Improved DPI handling. Druid should now redraw correctly when dpi changes. ([#1037] by [@rhzk])
+- winOS: Window created with OS default size if not set. ([#1037] by [@rhzk])
 - `Image` and `ImageData` exported by default. ([#1011] by [@covercash2])
 - `Scale::from_scale` to `Scale::new`, and `Scale` methods `scale_x` / `scale_y` to `x` / `y`. ([#1042] by [@xStrom])
 - Major rework of keyboard event handling ([#1049] by [@raphlinus])
@@ -222,6 +230,7 @@ Last release without a changelog :(
 ## 0.1.1 - 2018-11-02
 ## 0.1.0 - 2018-11-02
 
+[@rhzk]: https://github.com/rhzk
 [@futurepaul]: https://github.com/futurepaul
 [@finnerale]: https://github.com/finnerale
 [@totsteps]: https://github.com/totsteps
@@ -359,6 +368,7 @@ Last release without a changelog :(
 [#1062]: https://github.com/linebender/druid/pull/1062
 [#1072]: https://github.com/linebender/druid/pull/1072
 [#1081]: https://github.com/linebender/druid/pull/1081
+[#1037]: https://github.com/linebender/druid/pull/1037
 
 [Unreleased]: https://github.com/linebender/druid/compare/v0.6.0...master
 [0.6.0]: https://github.com/linebender/druid/compare/v0.5.0...v0.6.0

druid-shell/src/dialog.rs

@@ -32,15 +32,14 @@ pub enum FileDialogType {
 }
 
 /// Options for file dialogs.
+#[non_exhaustive]
 #[derive(Debug, Clone, Default)]
 pub struct FileDialogOptions {
     pub show_hidden: bool,
     pub allowed_types: Option<Vec<FileSpec>>,
     pub default_type: Option<FileSpec>,
     pub select_directories: bool,
     pub multi_selection: bool,
-    // we don't want a library user to be able to construct this type directly
-    __non_exhaustive: (),
 }
 
 /// A description of a filetype, for specifiying allowed types in a file dialog.

druid-shell/src/lib.rs

@@ -47,6 +47,7 @@ mod menu;
 mod mouse;
 mod platform;
 mod scale;
+mod screen;
 mod window;
 
 pub use application::{AppHandler, Application};
@@ -59,6 +60,8 @@ pub use keyboard::{Code, IntoKey, KbKey, KeyEvent, KeyState, Location, Modifiers
 pub use menu::Menu;
 pub use mouse::{Cursor, MouseButton, MouseButtons, MouseEvent};
 pub use scale::{Scalable, Scale, ScaledArea};
+pub use screen::Monitor;
+pub use screen::Screen;
 pub use window::{
     IdleHandle, IdleToken, Text, TimerToken, WinHandler, WindowBuilder, WindowHandle,
 };

druid-shell/src/platform/gtk/mod.rs

@@ -22,3 +22,4 @@ pub mod keycodes;
 pub mod menu;
 pub mod util;
 pub mod window;
+pub mod screen;

druid-shell/src/platform/gtk/screen.rs

@@ -0,0 +1,28 @@
+// Copyright 2020 The Druid Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+//! GTK Monitors and Screen information.
+
+use crate::screen::Monitor;
+use crate::kurbo::Size;
+
+pub(crate) fn get_display_size() -> Size {
+    log::warn!("Screen::get_display_size() is currently unimplemented for gtk.");
+    Size::new(0.0, 0.0)
+}
+
+pub(crate) fn get_monitors() -> Vec<Monitor> {
+    log::warn!("Screen::get_monitors() is currently unimplemented for gtk.");
+    Vec::<Monitor>::new()
+}

druid-shell/src/platform/gtk/window.rs

@@ -156,6 +156,18 @@ impl WindowBuilder {
         self.show_titlebar = show_titlebar;
     }
 
+    pub fn set_position(&mut self, _position: Point) {
+        log::warn!("WindowBuilder::set_position is currently unimplemented for gtk.");
+    }
+
+    pub fn maximized(&self) {
+        log::warn!("WindowBuilder::maximized is currently unimplemented for gtk.");
+    }
+
+    pub fn minimized(&self) {
+        log::warn!("WindowBuilder::minimized is currently unimplemented for gtk.");
+    }
+
     pub fn set_title(&mut self, title: impl Into<String>) {
         self.title = title.into();
     }
@@ -553,6 +565,36 @@ impl WindowHandle {
         }
     }
 
+    pub fn set_position(&self, _position: Point) {
+        log::warn!("WindowHandle::set_position is currently unimplemented for gtk.");
+    }
+
+    pub fn get_position(&self) -> Point {
+        log::warn!("WindowHandle::get_position is currently unimplemented for gtk.");
+        Point::new(0.0, 0.0)
+    }
+
+    pub fn set_size(&self, _size: Size) {
+        log::warn!("WindowHandle::set_size is currently unimplemented for gtk.");
+    }
+
+    pub fn get_size(&self) -> Size {
+        log::warn!("WindowHandle::get_size is currently unimplemented for gtk.");
+        Size::new(0.0, 0.0)
+    }
+
+    pub fn maximize(&self) {
+        log::warn!("WindowHandle::maximize is currently unimplemented for gtk.");
+    }
+
+    pub fn minimize(&self) {
+        log::warn!("WindowHandle::minimize is currently unimplemented for gtk.");
+    }
+
+    pub fn handle_titlebar(&self, _val: bool) {
+        log::warn!("WindowHandle::handle_titlebar is currently unimplemented for gtk.");
+    }
+
     /// Close the window.
     pub fn close(&self) {
         if let Some(state) = self.state.upgrade() {

druid-shell/src/platform/mac/mod.rs

@@ -25,3 +25,4 @@ mod keyboard;
 pub mod menu;
 pub mod util;
 pub mod window;
+pub mod screen;

druid-shell/src/platform/mac/screen.rs

@@ -0,0 +1,28 @@
+// Copyright 2020 The Druid Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+//! macOS Monitors and Screen information.
+
+use crate::screen::Monitor;
+use crate::kurbo::Size;
+
+pub(crate) fn get_display_size() -> Size {
+    log::warn!("Screen::get_display_size() is currently unimplemented for Mac.");
+    Size::new(0.0, 0.0)
+}
+
+pub(crate) fn get_monitors() -> Vec<Monitor> {
+    log::warn!("Screen::get_monitors() is currently unimplemented for Mac.");
+    Vec::<Monitor>::new()
+}

druid-shell/src/platform/mac/window.rs

@@ -147,6 +147,18 @@ impl WindowBuilder {
         self.show_titlebar = show_titlebar;
     }
 
+    pub fn set_position(&mut self, _position: Point) {
+        log::warn!("WindowBuilder::set_position is currently unimplemented for mac platforms.");
+    }
+
+    pub fn maximized(&self) {
+        log::warn!("WindowBuilder::maximized is currently unimplemented for mac platforms.");
+    }
+
+    pub fn minimized(&self) {
+        log::warn!("WindowBuilder::minimized is currently unimplemented for mac platforms.");
+    }
+
     pub fn set_title(&mut self, title: impl Into<String>) {
         self.title = title.into();
     }
@@ -838,6 +850,36 @@ impl WindowHandle {
     // TODO: Implement this
     pub fn show_titlebar(&self, _show_titlebar: bool) {}
 
+    pub fn set_position(&self, _position: Point) {
+        log::warn!("WindowHandle::set_position is currently unimplemented for Mac.");
+    }
+
+    pub fn get_position(&self) -> Point {
+        log::warn!("WindowHandle::get_position is currently unimplemented for Mac.");
+        Point::new(0.0, 0.0)
+    }
+
+    pub fn set_size(&self, _size: Size) {
+        log::warn!("WindowHandle::set_size is currently unimplemented for Mac.");
+    }
+
+    pub fn get_size(&self) -> Size {
+        log::warn!("WindowHandle::get_size is currently unimplemented for Mac.");
+        Size::new(0.0, 0.0)
+    }
+
+    pub fn maximize(&self) {
+        log::warn!("WindowHandle::maximize is currently unimplemented for Mac.");
+    }
+
+    pub fn minimize(&self) {
+        log::warn!("WindowHandle::minimize is currently unimplemented for Mac.");
+    }
+
+    pub fn handle_titlebar(&self, _val: bool) {
+        log::warn!("WindowHandle::handle_titlebar is currently unimplemented for Mac.");
+    }
+
     pub fn resizable(&self, resizable: bool) {
         unsafe {
             let window: id = msg_send![*self.nsview.load(), window];

druid-shell/src/platform/web/mod.rs

@@ -20,3 +20,4 @@ pub mod error;
 pub mod keycodes;
 pub mod menu;
 pub mod window;
+pub mod screen;

druid-shell/src/platform/web/screen.rs

@@ -0,0 +1,28 @@
+// Copyright 2020 The Druid Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+//! Monitor and Screen information ignored for web.
+
+use crate::screen::Monitor;
+use crate::kurbo::Size;
+
+pub(crate) fn get_display_size() -> Size {
+    log::warn!("Screen::get_display_size() is not implemented for web.");
+    Size::new(0.0, 0.0)
+}
+
+pub(crate) fn get_monitors() -> Vec<Monitor> {
+    log::warn!("Screen::get_monitors() is not implemented for web.");
+    Vec::<Monitor>::new()
+}

druid-shell/src/platform/web/window.rs

@@ -345,6 +345,18 @@ impl WindowBuilder {
         // Ignored
     }
 
+    pub fn set_position(&mut self, _position: Point) {
+        // Ignored
+    }
+
+    pub fn maximized(&self) {
+        // Ignored
+    }
+
+    pub fn minimized(&self) {
+        // Ignored
+    }
+
     pub fn set_title<S: Into<String>>(&mut self, title: S) {
         self.title = title.into();
     }
@@ -430,6 +442,36 @@ impl WindowHandle {
         log::warn!("show_titlebar unimplemented for web");
     }
 
+    pub fn set_position(&self, _position: Point) {
+        log::warn!("WindowHandle::set_position unimplemented for web");
+    }
+
+    pub fn get_position(&self) -> Point {
+        log::warn!("WindowHandle::get_position unimplemented for web.");
+        Point::new(0.0, 0.0)
+    }
+
+    pub fn set_size(&self, _size: Size) {
+        log::warn!("WindowHandle::set_size unimplemented for web.");
+    }
+
+    pub fn get_size(&self) -> Size {
+        log::warn!("WindowHandle::get_size unimplemented for web.");
+        Size::new(0.0, 0.0)
+    }
+
+    pub fn maximize(&self) {
+        log::warn!("WindowHandle::maximize unimplemented for web.");
+    }
+
+    pub fn minimize(&self) {
+        log::warn!("WindowHandle::minimize unimplemented for web.");
+    }
+
+    pub fn handle_titlebar(&self, _val: bool) {
+        log::warn!("WindowHandle::handle_titlebar unimplemented for web.");
+    }
+
     pub fn close(&self) {
         // TODO
     }

druid-shell/src/platform/windows/application.rs

@@ -22,14 +22,13 @@ use std::rc::Rc;
 
 use winapi::shared::minwindef::{FALSE, HINSTANCE};
 use winapi::shared::ntdef::LPCWSTR;
-use winapi::shared::windef::{HCURSOR, HWND};
+use winapi::shared::windef::{HCURSOR, HWND, DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2};
 use winapi::shared::winerror::HRESULT_FROM_WIN32;
 use winapi::um::errhandlingapi::GetLastError;
-use winapi::um::shellscalingapi::PROCESS_SYSTEM_DPI_AWARE;
 use winapi::um::winuser::{
     DispatchMessageW, GetAncestor, GetMessageW, LoadIconW, PostMessageW, PostQuitMessage,
     RegisterClassW, TranslateAcceleratorW, TranslateMessage, GA_ROOT, IDI_APPLICATION, MSG,
-    WNDCLASSW,
+    WNDCLASSW, SendMessageW
 };
 
 use crate::application::AppHandler;
@@ -38,7 +37,7 @@ use super::accels;
 use super::clipboard::Clipboard;
 use super::error::Error;
 use super::util::{self, ToWide, CLASS_NAME, OPTIONAL_FUNCTIONS};
-use super::window::{self, DS_REQUEST_DESTROY};
+use super::window::{self, DS_REQUEST_DESTROY, DS_HANDLE_DROPPED};
 
 #[derive(Clone)]
 pub(crate) struct Application {
@@ -64,10 +63,10 @@ impl Application {
     fn init() -> Result<(), Error> {
         // TODO: Report back an error instead of panicking
         util::attach_console();
-        if let Some(func) = OPTIONAL_FUNCTIONS.SetProcessDpiAwareness {
+        if let Some(func) = OPTIONAL_FUNCTIONS.SetProcessDpiAwarenessContext {
             // This function is only supported on windows 10
             unsafe {
-                func(PROCESS_SYSTEM_DPI_AWARE); // TODO: per monitor (much harder)
+                func(DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2);
             }
         }
         unsafe {
@@ -105,6 +104,11 @@ impl Application {
         unsafe {
             // Handle windows messages
             loop {
+                if let Ok(state) = self.state.try_borrow() {
+                    for hwnd in &state.windows {
+                        SendMessageW(*hwnd, DS_HANDLE_DROPPED, 0,0);
+                    }
+                }
                 let mut msg = mem::MaybeUninit::uninit();
                 let res = GetMessageW(msg.as_mut_ptr(), ptr::null_mut(), 0, 0);
                 if res <= 0 {