Solucionado
Quando fazemos o operador ternario
icon: icone != null ? Icon(icone) : null,
se o icone for nulo ele atribui null mas se o icone que passei for nulo eu passar o icone n é a mesma coisa que passar nulo no caso do Icon(icone) ou eu preciso fazer isso pq se nao eu to mandando um Widget que tem uma imagem nula e por isso ele ocupa espaço?
Mesmo o ícone não existindo, o espaço dele existe. O Icone é nulo, o espaço dele não. Ele está declarado no construtor, e por sua vez declarado no parâmetro da instância, porém não foi implementado. Por isso que em seguida o professor ensina como resolver essa situação pela flexibilização do widgets, fazendo isso pelo operador Ternário, eliminando aquele espaço decladrado no construtor.