Skip to content
This repository has been archived by the owner on Sep 24, 2024. It is now read-only.

Fix tel format and card section clickable type #31

Conversation

ShadowTourist
Copy link
Contributor

Add tel format and limitation for button tap in CardSectionView.

New Style
Screen Shot 2021-08-09 at 3 05 10 PM
Screen Shot 2021-08-09 at 3 05 32 PM
Screen Shot 2021-08-09 at 3 05 27 PM
Screen Shot 2021-08-09 at 3 06 39 PM
Screen Shot 2021-08-09 at 3 05 18 PM

@ShadowTourist ShadowTourist force-pushed the fix-tel-format-and-card-section-clickable-type branch from 203a77f to a4d7806 Compare August 9, 2021 07:22
@MarcoEidinger MarcoEidinger self-requested a review August 9, 2021 13:27
Copy link
Member

@MarcoEidinger MarcoEidinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@MarcoEidinger MarcoEidinger merged commit 395552d into SAP:dev Aug 9, 2021
@@ -133,6 +133,7 @@ extension CardSectionsView {

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ShadowTourist #if DEBUG is needed as PreviewData is only available in DEBUG. I pushed fix ad1cdd3 so no worries :) this is just a FYI for further use

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Thanks, I check the issues, this confused me before, I thought apple have fixed preview debug model issue.
But will fail release when preview use some Debug data only.

MarcoEidinger pushed a commit that referenced this pull request Aug 9, 2021
* fix: 🐛 trim all blank for tel url format

* fix: 🐛 only show link email and phone number as clickable
@ShadowTourist ShadowTourist deleted the fix-tel-format-and-card-section-clickable-type branch August 10, 2021 02:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants