LearnNewsExamplesServices
Frontmatter
id5728
titleworker.App: onOrientationChange() => limit the scope to affected windows
stateOpen
labels
enhancementno auto close
assigneesDinkh
createdAtAug 9, 2024, 12:04 PM
updatedAtMay 16, 2026, 10:50 PM
githubUrlhttps://github.com/neomjs/neo/issues/5728
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]

worker.App: onOrientationChange() => limit the scope to affected windows

Openenhancementno auto close
tobiu
tobiu commented on Aug 9, 2024, 12:04 PM

@Dinkh: adding an orientationchange notification for apps is a good idea in general.

however, the multi-window aspect probably should get honored.

tricky one: we could argue that the event can only happen on mobile where all windows are affected anyway.

edge-case: desktop, where you debug a specific window in the mobile view and switch the orientation there. in this case, other windows are not affected.

low prio.

tobiu added the enhancement label on Aug 9, 2024, 12:04 PM
tobiu assigned to @Dinkh on Aug 9, 2024, 12:04 PM
Dinkh
Dinkh Aug 10, 2024, 6:09 PM

@tobiu In fact this will only affect Desktop. And for desktop it seems more likely to use responsive settings than orientationchange.

For Tablets and Phone this should not matter, as you cannot have resizable windows.

I would argue, that this issue should be closed.

tobiu added the no auto close label on Oct 7, 2024, 11:52 PM