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

Fixing missing dev tools light mode send button #4403

Merged
merged 2 commits into from
Nov 3, 2021

Conversation

ouchadam
Copy link
Contributor

@ouchadam ouchadam commented Nov 3, 2021

Fixes #3674 Missing send icon in the light mode dev tools send event screens

Reusing the existing tint colour from the theme (matches the home screen search)

BEFORE LIGHT AFTER LIGHT
Screenshot_20211103_160921 after-light
BEFORE DARK AFTER DARK
Screenshot_20211103_160909 after-dark

@github-actions
Copy link

github-actions bot commented Nov 3, 2021

Unit Test Results

  62 files  ±0    62 suites  ±0   1m 4s ⏱️ +16s
118 tests ±0  118 ✔️ ±0  0 💤 ±0  0 ±0 
350 runs  ±0  350 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 2c39635. ± Comparison against base commit 3760401.

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.

Thanks a lot for this small fix!

@@ -9,6 +9,7 @@
tools:visible="true"
app:showAsAction="ifRoom"
android:icon="@drawable/ic_edit"
app:iconTint="?vctr_content_secondary"
Copy link
Member

Choose a reason for hiding this comment

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

Thanks for also having fixed that 👍
image

@@ -17,6 +18,7 @@
tools:visible="true"
app:showAsAction="ifRoom"
android:icon="@drawable/ic_send"
app:iconTint="?vctr_content_secondary"
Copy link
Member

Choose a reason for hiding this comment

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

I can never remember if it's works fine on API21, but I guess for a debug screen this is fine.

@bmarty bmarty merged commit d13e30b into develop Nov 3, 2021
@bmarty bmarty deleted the feature/adm/devo-submit-tint branch November 3, 2021 17:59
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.

dev tools are missing a button to submit send custom event and sent state event
2 participants