Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates; kas_view; #335

Merged
merged 28 commits into from
Jul 16, 2022
Merged

Updates; kas_view; #335

merged 28 commits into from
Jul 16, 2022

Conversation

dhardy
Copy link
Collaborator

@dhardy dhardy commented Jul 15, 2022

Dependencies updated: ron, usvg, resvg.

Dependency removed: stack_dst. Box<dyn ..> is a simpler alternative and there are no obvious performance issues, so I conclude that use of stack_dst was a premature optimization. (Also quite unsafe!)

#[widget { derive = self.foo; }] now supports partial trait implementations.

Bump MSRV to 1.62.0.

examples/sync-counter.rs: use only a Spinner widget (simpler code).

Revise drivers:

  • Rename DefaultViewView, DefaultNavNavView
  • Move event::Config impl to driver::EventConfig

Use Cargo features stabilized in 1.60.0:

  • Weak dependencies: serde no longer implies winit
  • Namespaced dependencies: rename dep_ron to ron using dep:ron
  • Namespaced dependencies: make features explicit, not implicit from optional dependencies
  • Replace canvas and svg features with tiny-skia and resvg, the latter including the Canvas widget; adjust how the import looks in kas doc

Crates:

  • Bump all versions to 0.11
  • New kas_view crate from the contents of kas::widgets::view; this is now exported as kas::view
  • Revise doc for model/view

dhardy added 28 commits July 14, 2022 08:24
This is premature optimization
Result: derive impls now support user-override of individual methods
Also fix message-handling of driver::Spinner/Slider
First, because we want a Copy type; second because it
actually stores a field we don't need.
This allows us to document available features and avoid some
unwanted implicit features. Requires rustc >= 1.60.
@dhardy
Copy link
Collaborator Author

dhardy commented Jul 16, 2022

CI failed on nightly due to: NotIntMan/inplace_it#10

@dhardy dhardy merged commit 42e6d8f into master Jul 16, 2022
@dhardy dhardy mentioned this pull request Jul 16, 2022
4 tasks
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