Field type text is deprecated, I would definitely use VARCHAR or NVARCHAR. ntext, text, and image (Transact-SQL)

Are you going to be comparing any other fields as well? As you're on 2016, have you considered Temporal (History) Tables instead of a trigger?

In a simple answer, yes, where new != old is fine, but thought best to bring up other solutions.