Entering a cruise altitude on PERF during APPR should re-establish the CLB/CRZ/DES flight phase

[beheh]

Aircraft Version

Development

Build info

{
  "built": "2022-03-13T12:33:48+00:00",
  "ref": "HEAD",
  "sha": "1c25455bb17e2a0015cb22c9d9c1ad299bf2bfa5",
  "actor": "crocket63 <[email protected]>",
  "event_name": "manual",
  "pretty_release_name": "master:1c25455b"
}

Describe the bug

Entering the cruise altitude on the PROG page during APPR has no visible effect.

Expected behavior

Entering a cruise altitude on the progress page during APPR should re-establish the CRZ flight phase (or CLB or DES where appropriate).

Steps to reproduce

1. Start a flight, spawn in the air
2. Set a cruise altitude
3. Enable the APPR phase
4. Enter a new cruise flight phase on the

References (optional)

While in DESCENT or APPR the CRZ entry should be dashed. Then, when entering a value, if it is at or below the FCU altitude while descending, or at or above the FCU altitude while climbing, the cruise altitude is updated, a scratchpad message NEW CRZ ALT XXXX is shown and the aircraft returns to the CRZ flight phase (or DES/CLB as appropriate)

Additional info (optional)

This was previously partially attempted in #6896.

Discord Username (optional)

No response

[derl30n]

We actually had this implemented at one point but voices raised that it shouldn't be possible. iirc pilots told us the only way to re-enter a different mode/phase from appr is by either activating alt dest or entering a new dest.

[beheh]

@derl30n Could you point me to that reference or maybe a Discord link? A search hasn't brought anything up for me. Especially seeing as #7101 is lined up now.

[derl30n]

This is a good question, we had this discussed with a lot of pilots since the documentation isn't that clear.

It must've been somewhere in pilot-feedback or in references. I'll see if I can find something.

[derl30n]

There is some reference that supports the pr's ref: https://discord.com/channels/738864299392630914/754851312306487326/817754295415865365

by the looks if it I've mistaken the go-around phase limitations with the approach phases ones.