FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(text): crash when removing FormattedText by vakrilov · Pull Request #7237 · NativeScript/NativeScript · GitHub

fix(text): crash when removing FormattedText - #7237

Merged
vakrilov merged 2 commits into
releasefrom
fix-formatted-string-parent-crash
May 13, 2019
Merged

vakrilov merged 2 commits into
releasefrom
fix-formatted-string-parent-crash

Conversation

Copy link
Copy Markdown
Contributor

PR Checklist

What is the current behavior?

See NativeScript/nativescript-angular#1818

What is the new behavior?

App no longer crashes

Fixes NativeScript/nativescript-angular#1818

ghost assigned vakrilov May 13, 2019
cla-bot Bot added the cla: yes label May 13, 2019
ghost added in progress and removed cla: yes labels May 13, 2019

Copy link
Copy Markdown
Contributor

test sdk_ng

cla-bot Bot added the cla: yes label May 13, 2019

Copy link
Copy Markdown
Contributor

test sdk_ng

Copy link
Copy Markdown
Contributor

test sdk_ng nativescript-angular#/tns-dist/nativescript-angular/PR/fac8bf7c5491842e48a15a06c86121a59ec58b8b/nativescript-angular-7.2.4.tgz

vakrilov merged commit 37b53c6 into release May 13, 2019
ghost removed the in progress label May 13, 2019
vakrilov deleted the fix-formatted-string-parent-crash branch May 13, 2019 14:03
manoldonev pushed a commit that referenced this pull request May 15, 2019
* fix(text): crash when removing FormattedText

* refactor: fixed crash on ios too

lock Bot commented May 20, 2020

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators May 20, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL