erisu commented on PR #157: URL: https://github.com/apache/cordova-plugin-dialogs/pull/157#issuecomment-1094184749
Can you fill out the PR description, following the template? And if it is to close a ticket also link the ticket. For example: Closes #<ticket number> Reviewing the code, I have a feeling that this is considered as a breaking change. I feel that it will change the current default visual to red. Anyone who updates a patch or minor and have a visual or behavioral change would not be expected. Can you please confirm if this is break? Could there be a flag that determines the visual? Or is there a way it can be a configuration flag where the supply the color of choice else default to current? One would expect red is generally the negative action but it is also the users decision to fit their color format.. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
