Drop iOS 7 support in readme #601
Comments
|
I guess we have to wait until Apple offically killed the iPhones which installed iOS 7 by default. Some people just won't upgrade iOS and use the apps, especially who paid for the app. At least I was told to support iOS 7 until it's dead. I got 5 iOS 7 devices online every day. |
|
What I'm saving is that it is possible to do so if you really want to but if you are a beginner and don't know much about swift or Xcode it is not easy. The other methods are way easier and many other libraries have moved to not supporting iOS7 for this very reason. See alamofire for example. It's just to reduce the number of issues we get for something that is really not about this library. It's an Xcode issue. |
|
What if some people start asking is it possible to support iOS 7? |
|
We say we don't officially support it but that it is possible. With maybe a link to apple docs about how to do it. |
|
I don't believe that this will change the amount of issues on the matter - it may even produce more of them... I think maybe we need to declare a specific date and time or a specific version where iOS 7 support will be dropped. When it is dropped - deployment target should also be raised to 8.0. |
|
the library deployment target is already at 8.0, I just saw it yesterday. |
|
It has to be since iOS 7 does not support dynamic frameworks. |
|
I am fine with your suggestion to give it a timeline. After iOS 10 maybe? |
|
Yes, i'd say the release day of iOS 10 is a great deadline :) |
|
Charts 3.0 release is a good deadline too, and close enough to iOS 10 ;-) |
migrate Chart v3 code in master to swift 2.3 syntax (#1387) * upgrade carthage to 0.17.1 * use master commit for iso-snapshot-test-case for now. once the fix is tagged we could change to the new version * change Rakefile, comment out tvOS because no FBSnapshotTestCase * Add missing imports for iOS 7 support * Update ChartSelectionDetail.swift * BUGFIX: fix infinite recursive call of getXIndex(_:) * for #1208, seems drawBarShadowEnabled should be false by default * 1. update ios-snapshot-test-case to 2.1.2 2. remove brew upgrade carthage, because travis saying * improve descriptions to warn users how to use setVisibleRange APIs * Typo fix * upgrade Realm to 1.0.2 * update Realm pod spec to 1.0.2 * fix pod spec typo * Work on x-values instead of x-indices (free for all :-) * Improvement for pie slice spacing calculation * Example improvements * Docs improvements * Major work on highlights * ChartMarker is now a protocol, and we have ChartMarkerView/Image helpers * Improvements regarding irrelevant inherited methods * Added feature to change color of "no data" text * More work on highlights * Viewport enhancements for better horizontal bar support * X-axis renderer enhancements for better horizontal bar support * Minor improvement to valueFormatter * Completed copy routine for scatter dataset * Pie: Renamed `drawSliceTextEnabled` to `drawEntryLabelsEnabled` * Pie: Added feature to offset center text * Pie: Separate color/font for entry labels (defaults to value color/font) * Minor improvements to demo * Minor refactoring of variable names * Example cleanup * Refactored scatter shape rendering into interface and renderers * Improved docs * Removed deprecated --var / var-- * Update README.md * Dropped iOS 7.0 support from README (Closes #601) * Fixed a RadarChart bug where value labels were misplaced * Realm fixes * Improvements to zoom methods * Legend improvements * DRYed some code * Adjusted maxHighlightDistance defaults * Removed NaN check for highlight. * Cleanup these comments * `drawBarShadowEnabled` is working again * Do not invalidate here by default, like other zoom methods * Simplified scatter shape enum model * These return a single pixel (x/y), for values (x value and y value) * Renamed x-pos to x-value where appropriate * Renamed ChartMarker -> IChartMarker * Namespace `Marker` names only for Objc (Swift 2.2) The Charts v3 changes are a chance to do this, as it's a breaking change anyway. * Do not force unwrap here * Fix for comments * More improvements to Markers, default behaviour constraint to view size * Docs * The highlight breaking changes are a chance to use @objc(name) feature * Use @objc(name) feature for rarely used classes (by users) * Improved entryIndex rounding method * Fixed drawing bounds for some of the charts * Removed unused yValueForXValue / yValuesForXValue * Set a default shape renderer * Take the correct barData object here * Fixed combined chart highlight with bar data * Minor refactoring * Removed the extra offset that messed up legend location * Cache XBounds object * Breaking change: `valueFormatter` now accepts an IValueFormatter This allows for formatting depending on viewport state etc. * Avoid x values in Pie's calcMinMax * This workaround is not required in the newer compilers * Renamed formatter interfaces FillFormatter -> IFillFormatter ValueFormatter -> IValueFormatter AxisValueFormatter -> IAxisValueFormatter * Avoid truncating using Int(...), it overflows * Corrected formula for 29-feb :-) * Some minor bug fixes for formatters * Polished demos * More improvements to examples * These should be internal * Fixed xAxis min/max calculation typo * Added helper to generate a fill formatter with a closure (block) * Added more samples * Updated tests * Set default barWidth to match old default behaviour * Renamed axisMinValue/axisMaxValue to axisMinimum/axisMaximum * Cleaned up header comments (I know that most of these files were created by me, I don't need that in the headers...) * Removed deprecated startAtZero properties * When only one of scaleXEnabled or scaleYEnabled is effective * Docs * Cleanup * Use EPSILON instead of an arbitrary value (Fixes #1239) * Fix dataset update in Dual line chart demo (Fixes #1250) * Add half line width to clipping rect of grid/limit lines (Fixes #1204) * Simplified legend entries configuration. Option to style dataset form. (Closes #1282) * Added feature for dashing legend line forms * Change default formLineWidth to match the default on MPAndroidChart * neededWidth should be enough here. formSize has no meaning globally. If more offset is needed - use extraOffsets, or adjust the maxSizePercent). * Renamed those classes too. Cleaner now. Should be too breaking... As with swift's minimal syntax - the chance of anyone referencing those by name is very low. On ObjC, the names stayed the same, for namespacing. * Take care of edge cases when calculating intervals (labelCount == 0) * autoScaleMinMax is working again (Fixes #1209) * Minor refactoring (Closes #1187) Sorry, un-mergable to `master`... * Fixed reversed condition from cbd7bf7 * It's clearer this way * Fill before stroke - because the fill may cover half of a thick stroke (Closes #982) * Added those new classes to tvOS/macOS targets * Migrated viewFromXib to macOS * Call `print` under `Swift` namespace, due to Cocoa ambiguity in NSView * Migrated macOS demos to Charts 3.0 * Use an optional for _displayLink (Fixes #1336) _displayLink might not have been created, i.e. in Interface Builder context. * Corrected LineChartRenderer cubicFill x range * Fixed horizontal cubic calculation bug introduced when removing x-index * Avoid using deprecated property in demo * Updated test screenshots * Animating the wrong axis for the starting point * migrate Chart v3 code in master to swift 2.3 syntax
iOS 7 is only a few percent of the market share right now and there are a lot of people having issues with how to install the library. Since this library is built with frameworks in mind we should consider dropping iOS 7 support since they aren't available for iOS 7.
Obviously it is still possible to use with iOS 7 but the only supported ways of installation would be the framework methods (import Xcode project, cocoapods, Carthage). This should maybe reduce all the issues we get about ios 7 installation.
The text was updated successfully, but these errors were encountered: