Skip to content

feat: Material progress indicator in projects list#1256

Merged
ilyassesalama merged 6 commits intoSketchware-Pro:mainfrom
SyntaxSpin:skinvent
Sep 21, 2024
Merged

feat: Material progress indicator in projects list#1256
ilyassesalama merged 6 commits intoSketchware-Pro:mainfrom
SyntaxSpin:skinvent

Conversation

@SyntaxSpin
Copy link
Contributor

No description provided.

@ilyassesalama ilyassesalama self-assigned this Sep 20, 2024
@ilyassesalama ilyassesalama added the UI/UX Related to user interface design or user experience. label Sep 20, 2024
@trindadedev13
Copy link
Collaborator

In my opinion, it didn't turn out very well, especially when the animation happens quickly.

@ilyassesalama
Copy link
Collaborator

In my opinion, it didn't turn out very well, especially when the animation happens quickly.

Yes, that's why I self-assigned this pull request to investigate better approaches before merging it.

@trindadedev13
Copy link
Collaborator

In my opinion, it didn't turn out very well, especially when the animation happens quickly.

Yes, that's why I self-assigned this pull request to investigate better approaches before merging it.

and what is the difference between assigning it or not? like, what does that change?

@SyntaxSpin
Copy link
Contributor Author

Ngl it's good and some guys can't see cause they have Myopia and hyperopia or the both at the same time so when you do an exact value for a loading indicator can cause UI issues for some people who make screen too big so it's good and enough and please close this discussion

@ilyassesalama ilyassesalama changed the title feat : adding new progress indicator in the loading app feat: Material progress indicator in projects list Sep 21, 2024
@ilyassesalama ilyassesalama merged commit 8867ab7 into Sketchware-Pro:main Sep 21, 2024
@trindadedev13
Copy link
Collaborator

Ngl it's good and some guys can't see cause they have Myopia and hyperopia or the both at the same time so when you do an exact value for a loading indicator can cause UI issues for some people who make screen too big so it's good and enough and please close this discussion

anyway it was changed to 100 and on my phone it was fine.

@ilyassesalama ilyassesalama added the merged This pull request was merged, it's only GitHub not showing it as merged. label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged This pull request was merged, it's only GitHub not showing it as merged. UI/UX Related to user interface design or user experience.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants