Code review comment for ubiquity:sil2100/screen-reader-target-system

Revision history for this message
Ɓukasz Zemczak (sil2100) wrote :

Fixed the commit message, thanks! As for your inline comment: the environment variable is only used for this very purpose (only for the screen reader), it's an env that's only shared between ubiquity and the casper hook. So I guessed that if orca is not installed, it anyway makes no sense to make the hook do anything. There is no risk of us disabling something that was supposed to be enabled.
Let me merge it then!

« Back to merge proposal