No. You missed to add the dots in the fqdn's in the data field. Something like ns1.domain.com is a fqdn (fully qualified domain name) and a fqdn has to end with a dot. So it must be ns1.domain.com. and mail.domain.com.
If you have relative records like "www", there is no dot at the end.
For example, your current MX record mail.domain.com. points to mail.domain.com.domain.com as it is currently a relative record and not a absolute.
I guess what you want is that emails for ...@domain.com gets delivered to the server mail.domain.com, the record for that will be: