Code review comment for lp:~jonas-drange/ubuntu-system-settings/about-dynamic-imei-fixes-1205294

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, small nitpick

> value: modemsSorted.length ? (imeiNumber || i18n.tr("None"))

the (imeiNumber || i18n.tr("None")) doesn't seem an obvious syntax to me, what would think of using
"value (modemsSorted.length && imeiNumber) ? imeiNumber : i18n.tr("None"))"
instead?

« Back to merge proposal