Você tem razão em apontar que os usuários são a coisa mais importante, no final. Mas aqui está o ponto que acho que você perdeu: outros desenvolvedores são usuários do seu código. É tão importante o que eles veem quanto os usuários do aplicativo veem.
Agora, se foi uma troca - se melhorar o código tornou a experiência do usuário pior -, então, devo dizer que preste atenção ao grupo maior (espero que seus usuários).
Mas não é uma troca. Você pode alcançar os dois objetivos simultaneamente. Então faça isso.
Como uma nota secundária, se você for o único desenvolvedor que trabalhará neste projeto (aplicativos descartáveis), não terá um problema imediato. Mas não seria melhor adotar o hábito de ser consistente agora do que quando você trabalha em equipe?
Há momentos em que uma equipe pode tomar a decisão de permitir que o código seja escrito 2 (ou até mais) maneiras diferentes, mas isso geralmente é feito com base em "vamos alterar nossa abordagem para todo o novo código, alterar código quando estamos nessa área, mas não alteramos o código que nunca é tocado até que sobra tão pouco que vale a pena fazer um projeto dele. " Esses casos precisam ser gerenciados com muito cuidado.
EDITAR : Para responder às suas edições: isso só torna isso pior, na minha opinião. Não é um caso de escrita desajeitada porque o IDE teria corrigido isso. Parece que havia outro bloco de código dentro do inner-if, que foi excluído junto com o outer-brace. Isso é tão fácil de corrigir que não há desculpa para não fazê-lo.
Depois que o recuo fica fora de controle, o código fica muito difícil de ler e ainda mais difícil de depurar. Eu não acho que você seja tão exigente.