search

schattenphoenix / es_applauncher

39 stars 1 forks source link

androidapps #8

Closed KoB-Kirito closed 4 weeks ago

KoB-Kirito commented 1 month ago

There is support for androidapps. There is a folder, the theme even has an image for it.

I can move apps from games over manually and it works fine, but everytime I have to add something new, es_applauncher breaks everything again.

Can you add a checkbox for apps too please?

image

schattenphoenix commented 1 month ago

There is support for androidapps. There is a folder, the theme even has an image for it.

I can move apps from games over manually and it works fine, but everytime I have to add something new, es_applauncher breaks everything again.

Can you add a checkbox for apps too please?

image

I don't understand If you check no checkbox it will put it in androidapps?

KoB-Kirito commented 1 month ago

Ahh, that's how that works 🤦🏻‍♂️

Never occured to me to check no checkbox at all, so never happened that anything got put in androidapps..

Maybe replace the disable checkboxes with App and disable apps that have no checkbox checked. Would make more sense imho.

schattenphoenix commented 4 weeks ago

Originally there was no check boxes There was only a list cause only apps existed (we added it manually). Then I added a disable on request, that's why it looks like this.

KoB-Kirito commented 4 weeks ago

I see. I'm just saying that it would be more logical now if every function was linked to an option. If I do nothing with an entry and then nothing is done with it, that would be easier to understand. At least that was not clear to me. It feels like a hidden function. Not consistent.

schattenphoenix commented 4 weeks ago

I get that Most people I've talked to about it basically were with me from the beginning so it makes sense for us lol.

I was actually hoping to not need to update it anymore cause the functionality will be integrated in ES-DE, but I don't know a time frame so it could be tomorrow, could be next year.

I might change it or at least clarify it in readme. Once I'm home next week

KoB-Kirito commented 4 weeks ago

If it gets integrated, it should be clear there though.

schattenphoenix commented 4 weeks ago

If it gets integrated, it should be clear there though.

I agree but it's a minor thing If I have time to change it, I might do so in the near future.

Since your issue was fixed, do you mind closing the issue?