OMI Apps - rework webview & pinned apps #1709

[Neotastisch]

Tests successful, see video below.

screen-20250118-191428.mp4

[Neotastisch]

also added app pinning & improved usability principals for the app page (better separation of individual apps)

[Neotastisch]

wont be able to complete the thumbnail task though

[Neotastisch]

note: cant test the featured apps since it doesnt show up for me. should be fixed though

[francip]

LGTM. Though I am not the best at Flutter layout, so take it with a grain of salt. :-)

[kodjima33]

@Neotastisch I need to see the updated demo video of how this looks like before merging

[Neotastisch]

@francip please do not approve changes, do not push this into production

@Neotastisch I need to see the updated demo video of how this looks like

Check telegram please

[beastoin]

hi man @Neotastisch , could you share the latest ver here. don't forget to double-check the requirements . keep the change minimal so that we are all on the same page; it's much easier for reviewing as well 💪

[Neotastisch]

Very busy, in a few hours maybe

…

-------- Ursprüngliche Nachricht -------- Am 21.01.25 14:05 um Thinh schrieb :

hi man ***@***.***(https://github.com/Neotastisch) , could you share the latest ver here. don't forget to double-check the requirements . keep the change minimal so that we are all on the same page; it's much easier for reviewing as well 💪 [Screenshot.2025-01-21.at.11.00.30.png (view on web)](https://github.com/user-attachments/assets/6124e018-20fb-42e0-8169-0fa5755e19f9) — Reply to this email directly, [view it on GitHub](#1712 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/ANCFYDBSZBD623CSJZ5O3QD2LXBOZAVCNFSM6AAAAABVNQB3Z2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBTGYYTEOBYGM). You are receiving this because you were mentioned.Message ID: ***@***.***>

[Neotastisch]

hi man @Neotastisch , could you share the latest ver here. don't forget to double-check the requirements . keep the change minimal so that we are all on the same page; it's much easier for reviewing as well 💪

Oh and I'm now doing that issue exactly, I'm not doing the thumbnails. Only the in-app changes.

[Neotastisch]

Tried but am not able to do backend chabges

[Neotastisch]

It's not a paid bounty anyway so should be alright

[Neotastisch]

screen-20250121-195206.mp4

(Ignore the two i icons, that was fixed)

[beastoin]

screen-20250121-195206.mp4

(Ignore the two i icons, that was fixed)

man, Neo. i know you are busy and i am happy when you have the obsession on seeing this happen on the omi app even though it's not a paid task. but i need you to deliver your high-quality work so that you can get paid.

i am sorry to say that this implementation is not good enough. i don't want to be a gatekeeper who always pointing out the flaws on the work of good guys who want to contribute and help build omi up. but you know that's also my responsibility to ensure the omi app works well and meets high standards. it's not about the code; it's about the product.

so, let's start with a good video that reflect all your best on this ticket. we will give feedback on it, again the app, not code. when we all agree that the app is good. i will take a look at the code and help you improve it.

for example:

1. the new setup steps UI is not better than the old one. what problem do you want to resolve ?

2. what's these icons ?

overall, it's super hard for me to imagine the flow of >rework webview & pinned apps. you should make it obvious for the user.

if you want to get paid, my advice is to make it obvious to the team that your work is a valuable and high-quality.

no worries, i'm a friend, who can help you improve your skill, not your boss.

@Neotastisch

[Neotastisch]

i do not need to be paid for this, i simply want that functionality myself.

1. i wanted to improve the design of the setup steps. It has the same functionality but is more self-explanatory that way
2. The top icons are:
   i: Switch between WebView and app details page (note: in the current code there is only one i, that was a bug when i recorded the video)
   pin: pin to top of apps page
   three dots (only when owner of the app): edit app & share
   share (only when not owner of the app): share

all of the icons below is the webview (so that one is the navigation bar in the brain app example)

i showed everything i changed in the video. i did not think it was worth it recording a video since i only changed the duplicate i's

[Neotastisch]

screen-20250125-134630.mp4