shouldn't you keep the readonly attribute on its own line for consistency with others ?