Skip to content

Conversation

@Simek
Copy link
Contributor

@Simek Simek commented Dec 17, 2025

Why

Spotted some warnings in console related to outline* style props which has been added in React Native 0.77 for better compatibility with CSS.

How

Add support for outlineStyle, outlineOffset, outlineWidth and outlineColor style props, add test spec covering various class values related to outline.

All possible classes, besides outline has been supported via dedicated resolvers. The stand-alone outline class has been supported via simple mapping to avoid complicating parsing. Please let me know if this is a correct approach!

According to Talwind docs, I have also added new keys (outlineOffset and `outlineWidth) for the theme customization.

A side note, blog post mentions old outlineSpread prop name which got later in time renamed to outlineOffset to align with web.

Refs:

@jaredh159
Copy link
Owner

sorry about the mess here, can you rebase this? i tried to do it, but i can't push to your branch, or i'm not clear how to resolve it myself.

…native-classnames into @simek/feat-support-outline-styles

# Conflicts:
#	src/UtilityParser.ts
@Simek Simek force-pushed the @simek/feat-support-outline-styles branch from 20890ea to 1507624 Compare December 19, 2025 15:53
@Simek
Copy link
Contributor Author

Simek commented Dec 19, 2025

No worries, I think it should be good now!

@jaredh159
Copy link
Owner

@Simek it's still showing conflicts in the github ui... 🤔

@Simek
Copy link
Contributor Author

Simek commented Dec 19, 2025

Hmm, that's interesting, I don't see any on my side 🤔

Screenshot 2025-12-19 172901

@jaredh159
Copy link
Owner

Screenshot 2025-12-19 at 11 58 58 AM

@Simek
Copy link
Contributor Author

Simek commented Dec 19, 2025

Indeed there is something off which has happen here after shipping other PRs, since main branch ref become fork/*.

Re-created the PR with the current master as HEAD, hope this solves the issue! Closing this one. 🙂

@Simek Simek closed this Dec 19, 2025
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.

2 participants