graphics for S60/P & TS101

[discip]

@Ralim
Here you go. 😀

Tried to implement all the necessary graphics for S60/P & TS101.

I know this will not work until you do all the necessities around it.
At least I hope the way I tried to implement this makes any sense. 😅

Not every graphic is pretty yet, but at least it should be testable.

Please let me know what you think.

menu animations
power
soldering
sleep
UI
advanced

statics
		S60/P	TS101
A-button
B-button
missing tip
unavailable
bootlogo	once
	loop

[discip]

@Ralim
Don't know if you had a chance to observe the changed pics.
I assume: Most likely not ! 😀
Anyway now there are attached gifs ...

Hope you like it.

[Ralim]

Amazing work, suppppppppeeerrrr appreciate it.

Haven't had a chance to test on hardware yet but when I get some downtime I will do some quick patching to get some of this in place asap 🤞

Thank you so much for this ❤️

[brushmate]

I somehow managed to compile your branch for my TS101 by rebasing your branch on the dev branch, but the icons are not being displayed correctly. I guess this is because support for bigger displays has not been added yet?

[Jonny-T-cyber]

Hi!
I'm waiting for the firmware for the TS101 to be completed so I can install it, in the meantime...
where can I find the ORIGINAL firmware for the TS101 so I can go back in case of problems?
Thanks

[discip]

@brushmate

I somehow managed to compile your branch for my TS101 by rebasing your branch on the dev branch, but the icons are not being displayed correctly. I guess this is because support for bigger displays has not been added yet?

As you may have noticed in your photo only a portion of the OLED but the individual graphics are intended for the whole screen.
So we have to wait until the support for the entire OLED gets implemented.

[Jonny-T-cyber]

On my new TS101 I have firmware V2.04.
While in the link you provided the latest version is V2.01 (which I had already found online).
Has anyone found a link to version V2.04?
Thanks

[ArturoGuerra]

Besides the graphics what else is needed? @Ralim

[Ralim]

This will be pulled in via the branch for splitting out rendering for different screen sizes. I'm slowly working on it as life allows me to.

[ArturoGuerra]

This will be pulled in via the branch for splitting out rendering for different screen sizes. I'm slowly working on it as life allows me to.

okay I'll have a look in a bit and see if I can help with that, also for the standardized power interface do you want to include all currently supported usb protocols so QC and USB-PD or just USB-PD?

[Ralim]

Once I finish splitting all the render/* functions out (in a day or two) I'll start a draft PR to track it.
When that happens then basically need to update every single render function to layout on the larger screen correctly 😓

I only intend to implement PD for all devices going forward if I get any choice in it. QC is a dead standard and its a shit-show for compatibility as well. Much prefer only focusing on standards that are open and properly standardised.

[discip]

@Ralim
Any news so far? 😊

[Ralim]

Any news so far? 😊

I've now pulled these into #1883, and will slowly be working over there to update all the drawing commands to draw the right size / location

[discip]

@Ralim
Hey Ben,
thank you for keeping this going. 😃👍

Could you please help finding the cause, why this doesn't compile for the S60P, but for the TS101 it does?

thanks in advance

[ia]

Could you please help finding the cause, why this doesn't compile for the S60P, but for the TS101 it does?

Hello.

It seems there is no declaration of ButtonA and related stuff in Font.h for S60P model explicitly. What's going on in Font.h is pure magic for me (I mean those constants), but apparently between all of these #ifdef MODEL / #else / #endif there is a missing conditional branch for macro pre-processor managing values of ButtonA and disconnectedTip.

Pre-processor goes here for S60P first, but then there is conditions like:
#ifdef MODEL_S60
and
#elif defined MODEL_TS101
but there is no values of the variables for S60P, because there is no such condition for S60P to define related values.

Basically, since support for S60P has been implemented, now Font.h should be updated and ported to support for S60P.

Probably for fix you should make this line to look something like this:
#if defined(MODEL_S60) || defined(MODEL_S60P)
but I'm not 100% sure that the values from S60 variables will be valid for S60P model.

Hope it will help.

[discip]

Thank you very much for the effort you put into describing the issue so precisely!

#if defined(MODEL_S60) || defined(MODEL_S60P)

You are absolutely right. That's the line I missed. 🤦‍♂️

[ia]

Thank you very much for the effort you put into describing the issue so precisely!

#if defined(MODEL_S60) || defined(MODEL_S60P)

You are absolutely right. That's the line I missed. 🤦‍♂️

I just was happy to help with everything I can, as always. Glad it worked. :)

[discip]

This was implemented in #1888.