Skip to content

Conversation

adrian-kong
Copy link
Contributor

@adrian-kong adrian-kong commented Mar 24, 2023

My.Movie.1.mp4

sped up by x4.
loader shows when clicking refresh button

ideally might change this to not use qml loaders, wanting to make this a single bar and do some lerping so it will look much cleaner (?)

trying to figure out how to best design this but sucks cant hotswap changes directly on console 🤮

@adrian-kong adrian-kong requested review from a team and silverjam April 3, 2023 10:07
@adrian-kong
Copy link
Contributor Author

Screen.Recording.2023-04-04.at.9.13.06.am.mov

think this looks way better. (will have to address the case when tcp connected but no settings etc... maybe just make this bar only on refresh action + write action?)

@swiftnav-travis
Copy link

Frontend and Release Workflow Started here

@adrian-kong adrian-kong changed the title add loader add loader [CPP-920] Apr 4, 2023
"Name": "",
"Value": ""
}]
rows: []
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need to remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this creates an empty row here:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removing it makes it like this:
image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@adrian-kong adrian-kong requested review from a team and silverjam April 6, 2023 06:46
@adrian-kong adrian-kong merged commit 3bc8828 into main Apr 6, 2023
@adrian-kong adrian-kong deleted the adrian/loader branch April 6, 2023 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants