rust-embedded / not-yet-awesome-embedded-rust

A collection of items that are not yet awesome in Embedded Rust
Creative Commons Attribution Share Alike 4.0 International
117 stars 11 forks source link

Update README.md #16

Closed flip111 closed 5 years ago

flip111 commented 5 years ago

This PR partially replaces https://github.com/rust-embedded/not-yet-awesome-embedded-rust/pull/6 Also added a second GUI library. And the PR uses the template in the readme.

rust-highfive commented 5 years ago

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @thejpster (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

jacobrosenthal commented 5 years ago

There is one existing library to build on, https://github.com/jamwaffles/embedded-graphics

flip111 commented 5 years ago

There is one existing library to build on, https://github.com/jamwaffles/embedded-graphics

@jacobrosenthal do you think we should mention that? I had a look at the readme, it seems to be for raw graphics and doesn't provide a toolkit with GUI elements. Maybe all that is needed is to reword the sentence i wrote a bit not to exclude this library from the wording ?

jacobrosenthal commented 5 years ago

Maybe, but also @jamwaffles might be a strong ally you might want to work with

On Wed, Jul 3, 2019 at 1:35 PM flip111 notifications@github.com wrote:

There is one existing library to build on, https://github.com/jamwaffles/embedded-graphics

@jacobrosenthal https://github.com/jacobrosenthal do you think we should mention that? I had a look at the readme, it seems to be for raw graphics and doesn't provide a toolkit with GUI elements. Maybe all that is needed is to reword the sentence i wrote a bit not to exclude this library from the wording ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rust-embedded/not-yet-awesome-embedded-rust/pull/16?email_source=notifications&email_token=AADPI5B4JQBKQWIF4TTMQMLP5UERZA5CNFSM4H5IQN4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZFTNIQ#issuecomment-508245666, or mute the thread https://github.com/notifications/unsubscribe-auth/AADPI5FOKFDFVK7OWIKYG6TP5UERZANCNFSM4H5IQN4A .

jamwaffles commented 5 years ago

it seems to be for raw graphics and doesn't provide a toolkit with GUI elements.

Correct, however one of my hopes is that it will be a library useful enough to build an embedded-ui crate on top of. I've had that in mind for a while, but haven't had a need for a UI library so never started one. I'm interested to see what people come up with!