on 02-22-2018 5:36 AM
If you use the FUIKPIProgressView inside a FUIKPIHeader it crash
I can see there is an internal property
even if I set it to true, it still crash
There is a lot of bugs in this SDK
please fix it
thanks
Alexandre,
Thank you for providing the code snippet. This issue has been fixed in the latest patch, could you kindly retry with it.
https://store.sap.com/sap/cp/ui/resources/store/html/SolutionDetails.html?pid=0000014485
Thanks,
Thasneem
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
thanks Andreas, sure, I used the code snippet from the SAP Mentor
var kpiProgressView: FUIKPIProgressView!
// KPI Position
kpiProgressView = FUIKPIProgressView(frame: CGRect())
kpiProgressView.chartSize = .small // Default
// KPI Content
kpiProgressView.isArrangedInKPIHeader = true
kpiProgressView.items = [FUIKPIMetricItem(string: "75"), FUIKPIUnitItem(string: "%")]
kpiProgressView.progress = 75 / 100
kpiProgressView.captionLabelText = "Completed"
kpiProgressView.tintColor = UIColor.preferredFioriColor(forStyle: .primary3)
kpiHeader = FUIKPIHeader(items: [kpiProgressView])
I am calling this in ViewDidLoad
after I do this
tableView.estimatedRowHeight = 98
tableView.rowHeight = UITableViewAutomaticDimension
tableView.backgroundColor = UIColor.preferredFioriColor(forStyle: .backgroundBase)
tableView.separatorStyle = .none
tableView.tableHeaderView = kpiHeader
tableView.delegate = self
tableView.dataSource = self
If you want also I could send you the Xcode project, for now I am just testing the SAPFiori framework with Mock Data inside a plist file
thanks
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
Alexandre,
Thanks for reporting this; can you please share the code snippet that you're using so I can forward this to the team for analysis?
Thanks
Andreas
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.
User | Count |
---|---|
80 | |
9 | |
9 | |
7 | |
7 | |
7 | |
6 | |
5 | |
5 | |
4 |
You must be a registered user to add a comment. If you've already registered, sign in. Otherwise, register and sign in.