Skip to content

Split up WindowDesc to allow sub-windows and window reconfiguration.#1235

Merged
rjwittams merged 1 commit intolinebender:masterfrom
rjwittams:pending-window
Sep 25, 2020
Merged

Split up WindowDesc to allow sub-windows and window reconfiguration.#1235
rjwittams merged 1 commit intolinebender:masterfrom
rjwittams:pending-window

Conversation

@rjwittams
Copy link
Collaborator

This is extracted from sub-windows.

It allows reconfiguration of windows and paves the way for construction of non-top level ones.

@rjwittams rjwittams requested a review from raphlinus September 14, 2020 12:33
@luleyleo luleyleo added the S-needs-review waits for review label Sep 15, 2020
Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

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

This looks good to me, a few questions inline but nothing blocking. Thanks!

@cmyr cmyr added S-ready PR is ready to merge and removed S-needs-review waits for review labels Sep 16, 2020
self
}

/// Set whether the window should be resizable.
Copy link
Member

Choose a reason for hiding this comment

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

What's the default?

self
}

/// Set whether the window should have a titlebar and decorations.
Copy link
Member

Choose a reason for hiding this comment

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

What's the default?

self
}

/// Sets the window position in virtual screen coordinates.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe mention that some platforms might not honor this.

@rjwittams
Copy link
Collaborator Author

I'll fix some bits later/tomorrow - a lot of the docs are just moved around from WindowDesc, I don't think fixing the content of those docs really has much to do with this PR. Some things are public because they need to be for subwindows I think (which has to construct windows slightly differently from the Druid side - ie not using WindowDesc).

@cmyr
Copy link
Member

cmyr commented Sep 18, 2020

my strategy around pub would be to make things non-pub now, and then change them to pub in the follow-up PRs, as they're needed; this is a nice forcing-function, so that we are only marking pub the things that need to be.

@rjwittams rjwittams merged commit ec7fa2c into linebender:master Sep 25, 2020
@rjwittams rjwittams deleted the pending-window branch September 25, 2020 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-ready PR is ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants