This will cause the screen reader to speak "dialog" twice, no? Because the "role" attribute already makes the screen reader speak the "dialog" role automatically.