maplibre / flutter-maplibre-gl

Customizable, performant and vendor-free vector and raster maps, flutter wrapper for maplibre-native and maplibre-gl-js (fork of flutter-mapbox-gl/maps)
https://pub.dev/packages/maplibre_gl
Other
226 stars 125 forks source link

docs: add docusaurus #461

Closed josxha closed 3 weeks ago

josxha commented 5 months ago

This pull request

A preview can be viewed here: https://josxha.github.io/flutter-maplibre-gl/

josxha commented 5 months ago

Not sure how to fix the dependency review CI.

Error: $GITHUB_STEP_SUMMARY upload aborted, supports content up to a size of 1024k, got 1975k.

The markdown summary exceeds the maximum size. Maybe because docusaurus adds too many dependencies at once?

josxha commented 1 month ago

I'm not sure if keeping all the docs for every version is a proper way to go, but why not.

This way it's possible to edited or improve old documentation as well as edit the latest version.

From the instructions, it is also not clear how to edit docs for an earlier version, for example.

Not sure how much documentation should get copied over from the docusaurus docs. They have documentaiton about versioned docs at https://docusaurus.io/docs/versioning. For users that just want to fix a typo or similar there is always the "Edit this page" button on every docs page.

josxha commented 1 month ago

I'm using this documentation page on the new maplibre package but if wanted we can still include it into this project, too.

josxha commented 1 month ago

Some additional notes: maplibre-gl-js had a discussion about the "right" static site generator in https://github.com/maplibre/maplibre-gl-js/issues/2150. In the end it's probably comes down to personal preference. I decided to go with docusaurus when creating this pr because of some features like versioned docs, an awesome site search and so on. As all documentation is in markdown, it would be possible to go with an other site generator, too.

The scope of this pull request is to provide a basic documentation setup. Adding full documentation is out of scope.

Because many npm dependencies are added in this pr, the Dependency Review CI fails.

josxha commented 3 weeks ago

Closing off this pull request now...