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

Export Image and ImageData by default #1011

Merged
merged 9 commits into from
Jun 2, 2020

Conversation

covercash2
Copy link
Contributor

this change exposes all of the Image functions and ImageData::empty without exposing the image crate. it also shows hints about the "image" feature in the API docs.

next

this PR doesn't add any functionality, really. still without the "image" feature enabled you can only draw an empty Image widget, but this is just an incremental step toward a more robust API for this feature.
i already have a bunch of documentation changes for image.rs, but i wanted to push this change first (i'm also working on my git habits).

This makes the `Image` widget and associated APIs
more discoverable.

There is also a note added that directs users on
how to enable the "image" feature.
@xStrom xStrom added the widget concerns a particular widget label Jun 2, 2020
Copy link
Member

@xStrom xStrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a good idea. Need to adjust some details though.

druid/src/widget/mod.rs Show resolved Hide resolved
druid/src/widget/image.rs Outdated Show resolved Hide resolved
druid/src/widget/image.rs Outdated Show resolved Hide resolved
druid/src/widget/image.rs Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
@xStrom xStrom added the S-waiting-on-author waits for changes from the submitter label Jun 2, 2020
@xStrom xStrom added S-needs-review waits for review and removed S-waiting-on-author waits for changes from the submitter labels Jun 2, 2020
Copy link
Member

@xStrom xStrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

The CI seems to have some problems right now, will merge once that gets resolved.

@xStrom xStrom removed the S-needs-review waits for review label Jun 2, 2020
@xStrom xStrom merged commit 8e46a06 into linebender:master Jun 2, 2020
@covercash2 covercash2 deleted the image_expose branch June 2, 2020 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
widget concerns a particular widget
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants