Skip to content
This repository has been archived by the owner on Aug 15, 2023. It is now read-only.

Add display support Proof Of Concept #117

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Conversation

elgopher
Copy link
Owner

@elgopher elgopher commented May 16, 2020

This PR is for experimentation with monitor support. The goal of it is to prototype new API and gather know-how needed to build production-grade API.

  • Add API for getting information about monitors
  • Open windows in fullscreen mode
  • Set fullscreen/windowed on existing window
  • Get and Set Window position

Some interesting findings:

  • changing window size (going fullscreen) is asynchronous
  • when resizing the window the size callback may not be called at all. Therefore method such as Resize must return an error after some timeout.
  • current video mode on MacOS X cannot be used for openning fullscreen window (the resulting window has a different size)

@elgopher elgopher self-assigned this May 16, 2020
@elgopher elgopher added the enhancement New feature or request label May 17, 2020
Copy link

@Nietworzy Nietworzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good code @jacekolszak!

@elgopher elgopher force-pushed the add-monitor-support branch 3 times, most recently from ac9cd6d to 4c4f35e Compare May 26, 2020 12:30
@elgopher elgopher changed the title Add monitor support Monitor support PoC Jun 20, 2020
@elgopher elgopher force-pushed the add-monitor-support branch 4 times, most recently from c142f52 to bae1a6b Compare July 5, 2020 15:50
@elgopher elgopher force-pushed the add-monitor-support branch 5 times, most recently from be317fc to 9bdc5f7 Compare July 23, 2020 17:54
@elgopher elgopher changed the title Monitor support PoC Add display support Jul 23, 2020
@elgopher elgopher force-pushed the add-monitor-support branch 12 times, most recently from 00354d6 to 19194ed Compare July 24, 2020 22:07
@elgopher elgopher force-pushed the add-monitor-support branch 3 times, most recently from 1e7bd3a to 1b71c96 Compare July 25, 2020 06:57
@elgopher elgopher force-pushed the add-monitor-support branch 3 times, most recently from bf6a42b to fb0f18d Compare July 25, 2020 23:36
@elgopher elgopher changed the title Add display support Add display support Proof Of Concept Jul 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants