Skip to content

Conversation

@mnaturel
Copy link
Contributor

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other :

Content

Replacing the color attribute vctr_unread_room_badge by vctr_content_secondary since it is relying on the same color number.

Motivation and context

See #5225

Screenshots / GIFs

Tests

  • Launch the application
  • Check notification badge is well displayed in home screen

Tested devices

  • Physical
  • Emulator
  • OS version(s): Android 11

Checklist

@github-actions
Copy link

Unit Test Results

  84 files  ±0    84 suites  ±0   53s ⏱️ -12s
157 tests ±0  157 ✔️ ±0  0 💤 ±0  0 ±0 
504 runs  ±0  504 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 4713fd9. ± Comparison against base commit 95b3afd.

@github-actions
Copy link

Matrix SDK

Integration Tests Results:

  • [org.matrix.android.sdk.session]
    passed=
  • [org.matrix.android.sdk.account]
    passed=
  • [org.matrix.android.sdk.internal]
    passed=
  • [org.matrix.android.sdk.ordering]
    passed=
  • [org.matrix.android.sdk.PermalinkParserTest]
    passed=

Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

@bmarty bmarty merged commit 5bd8247 into develop Feb 21, 2022
@bmarty bmarty deleted the feature/mna/5225-remove-color-resource branch February 21, 2022 15:49
@bmarty
Copy link
Member

bmarty commented Feb 21, 2022

Just one remark, could be nice to add in the description (for next time) something like closes #5225, so that the issue can be closed automatically by GitHub once the PR is merged.

@mnaturel
Copy link
Contributor Author

Just one remark, could be nice to add in the description (for next time) something like closes #5225, so that the issue can be closed automatically by GitHub once the PR is merged.

Okay I will keep that in mind for my next PR.

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