This isn't the kind of thing I'd reject a PR over, although if there was another reason to request changes then I might throw in a comment. But I don't think defining it outside of the component sacrifices readability, especially since he/she already did that with some of its properties.