New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: 🎸 [JIRA: HCPSDKFIORIUIKIT-2224] New FioriButtonStyle support #603
Conversation
API generated for binding property is not correct
Support FioriPrimaryButtonStyle, FioriSecondaryButtonStyle, FioriTertiaryButtonStyle
Should we let Andy review the header docs? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UA Review. Please see inline edits, comments, and questions.
I assume that "Fiori" should be capitalized so have changed all instances of "fiori" to "Fiori."
|
||
/// Creates a view that represents the body of a button. | ||
/// | ||
/// The system calls this method for each ``Button`` instance in a view |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double backticks?
Apps/Examples/Examples/FioriSwiftUICore/FioriButton/FioriButtonExample.swift
Outdated
Show resolved
Hide resolved
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
…nExample.swift Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
Co-authored-by: AndyRogersSAP <53194232+AndyRogersSAP@users.noreply.github.com>
} | ||
} | ||
|
||
fileprivate struct StatefulButton: View { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Private over fileprivate Violation: Prefer private
over fileprivate
declarations. (private_over_fileprivate)
* 'main' of github.com:SAP/cloud-sdk-ios-fiori: fix: 🐛 [bcp: 0] remove background in ObjectHeader (#614) docs: ✏️ build documentation with DocC and host on swiftpackageindex.com (#613) chore: 🤖 refactor FioriIcon and support accessing cgImage through Image (#612) feat: 🎸 jira 2286 sort & filter for SwiftUI project (#606) fix: 🐛 [bcp: 2370136916] fix width mode for DimensionSelector (#610) feat: [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons feat: [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons chore: 🤖 update CI job for watchOS to use xcode 15 (#609) chore: 🤖 run CI job on macos-13 and xcode 15.0 (#608) feat: [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons feat: [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons feat: 🎸 [JIRA: HCPSDKFIORIUIKIT-2224] New FioriButtonStyle support (#603) feat: 🎸 [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons feat: 🎸 [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons feat: [JIRA: HCPSDKFIORIUIKIT-2444] Update to include more icons
No description provided.