Content-Length: 315366 | pFad | https://github.com/NativeScript/NativeScript/pull/10399#event-10594600210

BC feat(css): text-stroke support by NathanWalker · Pull Request #10399 · NativeScript/NativeScript · GitHub
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.

Sign up for GitHub

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

Jump to bottom

feat(css): text-stroke support #10399

Merged
merged 4 commits into from
Oct 9, 2023
Merged

feat(css): text-stroke support #10399

merged 4 commits into from
Oct 9, 2023

Conversation

NathanWalker
Copy link
Contributor

@NathanWalker NathanWalker commented Oct 9, 2023 •
edited
Loading

PR Checklist

What is the current behavior?

You could not easily add stroke to text and core did not support text-stroke.

What is the new behavior?

CSS property text-stroke is now supported per this spec:
https://developer.mozilla.org/en-US/docs/Web/CSS/-webkit-text-stroke

Example:

<Label style="text-stroke: 1px black">text-stroke</Label>
Screenshot 2023-10-08 at 7 25 12 PM

closes #3597
closes #3972

@NathanWalker NathanWalker added this to the 8.6 milestone Oct 9, 2023
@cla-bot cla-bot bot added the cla: yes label Oct 9, 2023
@farfromrefug
Copy link
Collaborator

@NathanWalker I think the StyleableTextView should be native. By creating you actually made every single Label slower because it has o go through JS on every draw.
Seeing label is the most important(used) view, I think it s an issue

shirakaba
shirakaba reviewed Oct 9, 2023
View reviewed changes
Copy link
Contributor

@shirakaba shirakaba left a comment

Choose a reason for hiding this comment

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

Just some passing thoughts while eating a donut. I haven't looked too closely beyond the bits I've commented on.

@NathanWalker
Copy link
Contributor Author

@NathanWalker I think the StyleableTextView should be native. By creating you actually made every single Label slower because it has o go through JS on every draw. Seeing label is the most important(used) view, I think it s an issue

Good call @farfromrefug I’ll make it native.

@NathanWalker NathanWalker merged commit d647823 into main Oct 9, 2023
@NathanWalker NathanWalker deleted the feat/css-text-stroke branch October 9, 2023 19:00
tdermendjiev pushed a commit to tdermendjiev/NativeScript that referenced this pull request Oct 25, 2023
@NathanWalker @tdermendjiev
feat(css): text-stroke support (NativeScript#10399)
07fc360 
closes NativeScript#3597
closes NativeScript#3972
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shirakaba shirakaba shirakaba left review comments

Assignees
No one assigned
Labels
cla: yes hacktoberfest hacktoberfest-accepted
Projects
None yet
Milestone
8.6
Development

Successfully merging this pull request may close these issues.

Feature request: text-stroke property and/or text-shadow [Feature] add outline to label and button text
3 participants
@NathanWalker @farfromrefug @shirakaba








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/NativeScript/NativeScript/pull/10399#event-10594600210

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy