Skip to content
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

Multi line tag support #262

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pritam-bs
Copy link

[Description]:

  • Support for byWordWrapping
    Simulator Screen Shot - iPhone SE (2nd generation) - 2020-07-31 at 10 14 35

[Description]:
- Support for byWordWrapping
@pritam-bs pritam-bs changed the title [Title] Multi line tag support Multi line tag support Jul 31, 2020
@jsyrtsov
Copy link

@pritam-bs
Hello :)
Can you help me please?
I'm trying to use your solution for multiline tags, but unfortunately I'm catching fatal error.
Fatal error: Negative Array index is out of range

@pritam-bs
Copy link
Author

@jsyrtsov
Hi!
Please provide some details.

@jsyrtsov
Copy link

@pritam-bs
Thank you for your reply ! :)
Do you have any chat or messenger ?

@khanskt786
Copy link

khanskt786 commented Nov 3, 2020

@pritam-bs I also tried your solution and it throws fatal error, here are the logs

Fatal error: Array index is out of range: file /Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-1103.2.25.13/swift/stdlib/public/core/Array.swift, line 382
2020-11-03 15:04:23.466606+0500 LLAXA[30589:293613] Fatal error: Array index is out of range: file /Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-1103.2.25.13/swift/stdlib/public/core/Array.swift, line 382

Crash occures in this function

@discardableResult
open func addTagViews(_ tagViewList: [TagView]) -> [TagView] {
defer { rearrangeViews() }
tagViewList.forEach {
tagViews.append($0) // on this line to be precise
tagBackgroundViews.append(UIView(frame: $0.bounds))
}
return tagViews
}

-------------Solution----------
Clean build folder delete derived data and run, project is now working crash free but no multiline text added its still the same

Copy link

@maybekylecodes maybekylecodes left a comment

Choose a reason for hiding this comment

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

Tried this out and this worked well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants