Skip to content

Commit e6596d0

Browse files
committed
Rework the Multi window support long term plan in the readme as I finally found a way to do it that I like.
1 parent 2c0d7fb commit e6596d0

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

README.md

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -234,22 +234,23 @@ It should also look at least acceptable.
234234

235235
Add multi window support for the engine.
236236

237-
Should oly add very little code and practically zero complexity.
237+
Being able to open more windows should only add very little code and practically zero complexity by itself.
238238

239239
The issue is popups. Inheriting them from a Window (Viewport) class is incredibly inefficient,
240240
and custom handling for them will be extremely messy. A good solution for this is going to be needed.
241241

242-
Currently my best idea is to have 2 sets of popups, and the user can decide whether to have multi window support
243-
or not by using the proper ones. Still trying to figure out a way to make this simpler.
242+
Likely the proper solution:
244243

245-
Or the ClassDB could also switch between classes similar to how compat classes work.
246-
This does have issues.
244+
Have an another set of popups in an engine module, so they can be disabled if not used.
247245

248-
Or a pseudo-viewport + maybe pesudo Window class could be added, and just make all the code handle it.
246+
WindowedPopup etc...
249247

250-
Could also be inherited from Viewport or viewport could inherit from it, but it's ignored if disabled, also it does not allocate anything if disabled.
248+
Have them inherit from Window, if windowing is enabled, and Control if it's disabled,
249+
So no performance penalty on platfroms with no windowing.
251250

252-
Or Viewports could have an enabled property or similar, but that's prbably going to cause issues.
251+
Could force windowing support for them to be disabled at compile time even if a platform can do windowing.
252+
253+
Their api does not need to match the built in popups exactly.
253254

254255
### WebNodes
255256

0 commit comments

Comments
 (0)