Skip to content

Screen Flow: Add "stack_push + attach" to toggle action #185

Description

@parsley

Is your feature request related to a problem? Please describe.
Yes, but think I'll write a separate issue for that and link it later.
(Brief summary: GX_ACTION_TYPE_SCREEN_STACK_PUSH action does not check return value of gx_system_screen_stack_push(). That discovery lead to this feature request.)

Describe the solution you'd like
The GX_ACTION_TYPE_TOGGLE Screen Flow action already allows to "toggle backwards" to return to the top most screen on the screen_stack and detach the current screen.
My feature request is to add the equivalent combination in forward direction: Push the current screen to the screen_stack and attach a new target (but only if the push operation was successful! ;-) ) together in a single GX_ACTION_TYPE_TOGGLE.

Describe alternatives you've considered
On a single trigger I did two actions: GX_ACTION_TYPE_ATTACH and GX_ACTION_TYPE_SCREEN_STACK_PUSH. This lead to the issue of multiple simultaneously attached screens because once the screen_stack was full gx_studio_auto_event_handler() silently fails pushing&detaching but happily attaches new screens.
Since GX_ACTION_TYPE_SCREEN_STACK_PUSH does neither check nor forward the return value of gx_system_screen_stack_push() a workaround is not obvious to me.

Additional context
I hope my request is understandable and not some weird use case just me alone came across?

(At the time of issuing this ticket I used GUIX release v6.5.1.202602a)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    featureNew feature or enhancement request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions