Skip to content

fix+docs: P0/P1 behaviour fixes and 1.0.0 doc alignment - #1

Merged
zorrooz merged 1 commit into
mainfrom
dev
Sep 19, 2026
Merged

zorrooz merged 1 commit into
mainfrom
dev

Conversation

@zorrooz

@zorrooz zorrooz commented Sep 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Behaviour fixes (P0/P1): mark_bar scale preservation, nested compose annotations, compose_annot gap, label hide sentinel, errorbar/ribbon width, colour/fill legend twin, make_mark name, mark_map palette, mark_rule linewidth, style base_theme, compose default sizes / inset guides
  • Docs: NEWS 1.0.0, README maturing, AGENTS/API/pkgdown aligned
  • Hygiene: lintr 0, tests use plotit::: for internals

Verification

  • Full testthat suite: 0 Failed / 0 Error
  • lintr::lint_package(): 0 issues

Behaviour:
- mark_bar no longer replaces a pre-installed continuous y scale
- nested compose bakes inner composite annotations before assembly
- compose_annot(gap=) keeps spacer rows out of the base cell
- label_* hide sentinel is isFALSE(); text='FALSE' stays text
- errorbar/ribbon width injected only for geoms that accept it
- colour/fill mirroring attaches the twin scale with guide='none'
- make_mark passes mark_name into ._register_mark_method
- mark_map layer colour/fill uses the curated default palette
- mark_rule segment path shares the hline linewidth default
- style() accepts theme functions; warns when font args clash with a theme object
- compose default sizes honour present sides, annot gaps, and design grids
- composite inset legend parking uses patchwork '&' when needed

Docs / hygiene:
- NEWS.md 1.0.0 entry; README lifecycle maturing; AGENTS/API/pkgdown aligned
- export dpi documented as 600; plotit defaults 89x56 mm in api.Rmd
- lintr clean; tests use plotit::: for internals; root artifacts removed
@zorrooz
zorrooz merged commit 9f5721b into main Sep 19, 2026
14 checks passed
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.

1 participant