Skip to content

Custom terminal shell setting cannot be selected #21586

Closed

Description

@mingdaoai

Problem

On macOS systems whose configured system shell is /bin/tcsh, Settings → Terminal displays the System shell option, but clicking Custom shell does not activate the custom-shell input.

Expected

Selecting Custom shell should reveal an editable shell path field.

Observed

The selection remains on System shell and no input appears.

Environment

  • macOS
  • Orca local build

A regression test and fix are prepared on branch fix-custom-terminal-shell-setting.

Activity

  1. added theissue type on Sep 19, 2026
  2. AmethystLiang commented on Sep 21, 2026

    @AmethystLiang
    Contributor

    thanks for reporting. looking into it

  3. Ninerian commented on Sep 23, 2026

    @Ninerian

    The cause is

    : { terminalDefaultShell: configuredShell || systemShell }

    The value of customShell is initially an empty string which is the same as if the system shell is chosen. Therefore the logic stays in the system shell path.

  4. nwparker commented on Oct 22, 2026

    @nwparker
    Contributor

    Verified this is implemented on current main (4dd97f02d5e240ab5d756017a946f8a9617678c6). Selecting Custom shell now writes configuredShell || systemShell, so an initially empty custom setting becomes /bin/tcsh and the editable path field appears. This replaces the empty-string selection described above.

    Evidence: TerminalPane.tsx; the implementation is included in merged #25078 (dd39dcba5f99710251bac1a33392bb41a1fd3945, ancestor of main). #21587 itself was not merged, so this closure relies on the current implementation.

    I also ran a temporary React component test against the actual TerminalPane, with the platform shell set to /bin/tcsh and an initially empty setting: selecting Custom shell selected its radio and revealed /bin/tcsh; editing the field changed its value; switching back to System shell hid the field. 1 test passed with ORCA_BACKGROUND_LAUNCH=1 pnpm test …. This was a component test, not a desktop app run.

    Closing as completed because the full reported selection/input behavior is now delivered.

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

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions