Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New design #96

Merged
merged 43 commits into from
Jul 3, 2024
Merged

New design #96

merged 43 commits into from
Jul 3, 2024

Conversation

mdrlzy
Copy link
Member

@mdrlzy mdrlzy commented May 18, 2024

No description provided.

@mdrlzy mdrlzy changed the title New design New design [WIP] May 18, 2024
@mdrlzy mdrlzy marked this pull request as ready for review June 13, 2024 23:59
@mdrlzy mdrlzy requested a review from kirillt June 14, 2024 00:07
@mdrlzy mdrlzy changed the title New design [WIP] New design Jun 14, 2024
@kirillt kirillt requested a review from hieuwu June 15, 2024 07:46
@kirillt
Copy link
Member

kirillt commented Jun 15, 2024

@shubertm please mark this PR with new label QA passed when all issues are either resolved, or tracked in the issue tracker.

@shubertm
Copy link
Member

shubertm commented Jun 16, 2024

Screenshot_20240616_195009_dev.arkbuilders.rate.jpg

  • The status bar color should be well designed

Screenshot_20240617_104221.jpg

  • Some margin should be added before the icon here

Screenshot_20240617_104150.jpg

  • We should have some style for ripples on the link button too
SVID_20240616_203207_1.mp4
  • The current price decreases suddenly with decimal point moving a few steps left when Every button is clicked

@Query("SELECT * FROM RoomAsset")
fun allFlow(): Flow<List<RoomAsset>>

@Query("DELETE FROM RoomAsset where id = :id")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
@Query("DELETE FROM RoomAsset where id = :id")
@Query("DELETE FROM RoomAsset WHERE id = :id")


@Dao
interface PortfolioDao {
@Insert(onConflict = OnConflictStrategy.REPLACE)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This could be replaced with @Upsert annotation

Suggested change
@Insert(onConflict = OnConflictStrategy.REPLACE)
@Upsert

@Insert(onConflict = OnConflictStrategy.REPLACE)
suspend fun insert(pairAlert: RoomPairAlert): Long

@Query("SELECT * FROM RoomPairAlert where id = :id")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
@Query("SELECT * FROM RoomPairAlert where id = :id")
@Query("SELECT * FROM RoomPairAlert WHERE id = :id")

@mdrlzy
Copy link
Member Author

mdrlzy commented Jun 19, 2024

The current price decreases suddenly with decimal point moving a few steps left when Every button is clicked

@shubertm
This is intentional. There it is considered 10% of current price in Every tab. You don't want to be notified when the price increases by 100% (this is unlikely to happen)

mdrlzy added 26 commits July 3, 2024 17:49
filter feature, create new pair from existed pair and code
fix bottom nav design
Firebase setup, track screen usage, remove ACRA


revert fix Webview data directory suffix crash (caused by ACRA)


ui fixes
Fix empty screen btn padding
white navigation bar


intercept clicks on snackbar background


sort CurrencyName by code in repo


Quick item second icon fix


fix fiat icons


ui fix: center currency name if no description


discord link
@kirillt kirillt merged commit 3245bc6 into main Jul 3, 2024
2 checks passed
@kirillt
Copy link
Member

kirillt commented Jul 3, 2024

Great work! 💪

Thanks @mdrlzy and also @hieuwu and @shubertm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants