not-jan / apex-tux

Linux support for the OLED screen of the Apex Pro, Apex 5 and Apex 7 keyboards by SteelSeries
The Unlicense
68 stars 14 forks source link

Gif implementation #34

Closed jehendeoff closed 1 year ago

jehendeoff commented 1 year ago

Hi, I tried to implement gifs as both : a provider and a renderer, so that it could potentially easily integrated in other providers.

If you see anything that I should change, please don't hesitate, I'm still a beginner in rust.

Please note before approving that this is an experiment, and in no way should it be the last version to hit production.

not-jan commented 1 year ago

Currently not at home, I'll keep an eye on my mails. Feel free to ping me when you're read for a review :) Thank you for contributing!

jehendeoff commented 1 year ago

Hi @not-jan, I think I finished implementing the gif renderer and provider, I'd like if you could take a look at the code and tell me if anything seems wrong

Socerest2 commented 1 year ago

Hey, I'm getting the following compilation error when I try to build: E0635 I'm not that into rust so I can't really help you out there but it says "The #![feature] attribute specified an unknown feature."

jehendeoff commented 1 year ago

Hi @Socerest2, that's weird, I could compile it yesterday, could you tell me what's your OS, your rust version, and the full output of the fail? and, does it also fails with the main branch? (if so, it seems a feature was removed from rust nightly)

jehendeoff commented 1 year ago

Hey, I'm getting the following compilation error when I try to build: E0635 I'm not that into rust so I can't really help you out there but it says "The #![feature] attribute specified an unknown feature."

do you still have your issue ? I managed to compile it just today, so I think you have a problem on your end, and I think your issue should also be present when compiling the main branch? correct me if I'm wrong please

Socerest2 commented 1 year ago

Hey Sorry! I have a bad habit of not checking github sometimes! I still have the error.

jehendeoff commented 1 year ago

I think I did everything that you asked for, if you see anything new that's wrong, please tell me, I'll gladly fix it

not-jan commented 1 year ago

Merged, thank you so much! :)