Skip to content

Conversation

@LHBruneton-C2C
Copy link

@LHBruneton-C2C LHBruneton-C2C commented Jun 5, 2025

Description

Implements #841

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Dependency updates
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe)
  • I am unsure (we'll look into it together)

Do you introduce a breaking change?

  • Yes
  • No
  • I am unsure (no worries, we'll find out)

Checklist

  • I understand and agree that the changes in this PR will be licensed under the BSD 2-Clause License
  • I have followed the guidelines for contributing
  • The proposed change fits to the content of the code of conduct
  • I have added or updated tests and documentation, and the test suite passes (run npm test locally)
  • I'm lost; why do I have to check so many boxes? Please help!

TO REVIEW

  • Confirm FlatStyleArray as Symbolizers
  • Confirm that rotation needs / Math.PI * 180
  • Confirm when to use or not use structuredClone

TODO

  • dynamic text label
  • unsupported properties
  • property-based filter
  • boolean function
  • case function
  • nested function
  • pi <=> 3.141592653589793
  • distinguish simple stroke from fill with outline

@LHBruneton-C2C LHBruneton-C2C force-pushed the codesprint/flat-style-write branch 6 times, most recently from 71f0930 to b19b780 Compare August 25, 2025 14:24
@jahow jahow force-pushed the codesprint/flat-style-write branch from 9a9ad28 to 90d48b3 Compare August 26, 2025 14:19
@jahow jahow changed the title Codesprint: flat style write Add complete support for OpenLayers flat style format Aug 26, 2025
@jahow jahow force-pushed the codesprint/flat-style-write branch from e27126a to 012e4e0 Compare September 1, 2025 12:55
@jahow jahow force-pushed the codesprint/flat-style-write branch from 012e4e0 to c88f8e0 Compare September 1, 2025 13:06
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