Would it make more sense to define this on `GridDialect` instead?