breautek commented on issue #212: URL: https://github.com/apache/cordova-plugin-device/issues/212#issuecomment-3025458687
I don't think this should be considered a bug, but I'll leave this issue open in case my opinion is the odd one out. I'll explain my rationale though: Our [docs](https://github.com/apache/cordova-plugin-device?tab=readme-ov-file#deviceversion) states: > Get the operating system version. which is similar language to the Apple documentation on [UIDevice systemVersion](https://developer.apple.com/documentation/uikit/uidevice/systemversion?language=objc). `[UIDevice systemVersion]` being the API used to fetch the version string: https://github.com/apache/cordova-plugin-device/blob/2c6219fd1e05eccd6817e79ed51faabbd8415891/src/ios/CDVDevice.m#L94 I understand it might not be very intuitive obvious but it's Apple's decision to stop using the OS version as their marketing name. The version should continue to represent the OS version, as signaled from the iOS SDK. Changing the behaviour to conditionally check for OS >= 19 would be a breaking change to any one who decided to handle mapping the OS version to the marketing name, and would deviate from the documentation. If the version is being used to display information in the UI, I think it should be responsibility of the application to map the OS version to the marketing name where necessary. Keeping the version attribute as the logical OS version. -- 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: issues-unsubscr...@cordova.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org
