Code review comment for lp://qastaging/~elementary-pantheon/scratch/confirm-heavy-files

Revision history for this message
Danielle Foré (danrabbit) wrote :

I'm not sure this is a great metric. For example the Scratch binary is under 1MB as are a number of other apps. It catches the one case, I dunno if it completely solves the issue of dropping in a binary.

I'm also not sure AlertView is the right widget for this. There's no obvious way to cancel.

« Back to merge proposal