Skip to content

Make StylingPolicy public - #339

Closed
EugenMayer wants to merge 1 commit into
OWASP:mainfrom
KontextWork:feature/public-styling-polic
Closed

Make StylingPolicy public#339
EugenMayer wants to merge 1 commit into
OWASP:mainfrom
KontextWork:feature/public-styling-polic

Conversation

@EugenMayer

Copy link
Copy Markdown

See #132 for the motivation.

Did not use a builder since having access to StylingPolicy.apply can be very critical

Did not use a builder since having access to StylingPolicy.apply can be very critical
@EugenMayer

Copy link
Copy Markdown
Author

Any chance to have a opinion / review for this one? Thanks for the effort

@EugenMayer

Copy link
Copy Markdown
Author

Maybe we could at least get an answer on this this year? :) Thanks!

@jmanico

jmanico commented Sep 7, 2026

Copy link
Copy Markdown
Member

@EugenMayer sorry this sat unanswered for so long. The use case is a good one and we do want to take it — I've written up a proposed shape in #381 (a public factory on CssSchema rather than making StylingPolicy itself public, plus a note on how url() values must be handled outside the builder). If you're still interested, a revised PR against #381 with tests would be very welcome; otherwise we'll pick it up. Thanks for your patience.

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