kitty.shellIntegration: document that shells need to be enabled in home manager

[ksyx]

Description

Currently, programs.kitty.shellIntegration.enable*Integration seemed to require respective shells to be enabled. With its default calculated by !(elem "disabled" (splitString " " config.programs.kitty.shellIntegration.mode)), which seemed to be true for all shells by default if integration mode is set to something enabling. Hence, to reduce confusion of not getting the integration after enabling, it may worth adding a note that these options depends on programs.<shell>.enable=true to work, in the descriptions of these options.

[ksyx]

@Scrumplex Please see if this is worth adding

[Scrumplex]

I think the conditionals should be somewhat flipped for those options. Maybe set their default value to programs.<shell>.enable and just make it a noop if shell integration is disabled.

[ksyx]

Oh this sounds even better!

[rigille]

Thanks! I was so confused about why shell integration wasn't working before reading this issue.