[PR #828] Update Display.cs to current spec #1315

New Issue

claunia · 2026-01-29T16:59:34Z

claunia commented

2026-01-29 16:59:34 +00:00

Original Pull Request: https://github.com/ElectronNET/Electron.NET/pull/828

State: closed
Merged: Yes

As you can see here, the current Display.cs file in Electron.NET is not up to date. I've added all the missing fields (except for those which show up in the docs, but do not actually appear in the object we fetch from electron), and checked locally to see that it works.

One thing to note is that I did not change the Id value from string to int, because that caused the Electron.Screen.GetAllDisplaysAsync() method to hang forever, and changing it also means breaking backwards compatibility.

**Original Pull Request:** https://github.com/ElectronNET/Electron.NET/pull/828 **State:** closed **Merged:** Yes --- As you can see [here](https://www.electronjs.org/docs/latest/api/structures/display), the current Display.cs file in Electron.NET is not up to date. I've added all the missing fields (except for those which show up in the docs, but do not actually appear in the object we fetch from electron), and checked locally to see that it works. One thing to note is that I did not change the `Id` value from `string` to `int`, because that caused the `Electron.Screen.GetAllDisplaysAsync()` method to hang forever, and changing it also means breaking backwards compatibility.

claunia added the pull-request label 2026-01-29 16:59:34 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/Electron.NET#1315