While it certainly breaks the logic/design separation ideal, so long as it works and can be modified if need be - (never say never) - and is acceptable within any other constraints then why not?

However, it doesn't have any dynamic variables in so it could simply be written as an HTML form. I assume it's encapsulated further with a start_html() ... end_html() somewhere?