fix: sketch general issues (WPB-8810) (WPB-8811) (WPB-8812)

[yamilmedina]

WPB-8810 [Android] Sketcher Undo button works on 3rd tap

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Issues

• Color picker, didn't consider the status bar padding
• When Keyboard open, the drawing showed a loop collapsing keyboard and no drawing mode.
• Undo button took at least 3 taps to perform action.
• Darkmode title not visible

Solutions

• Add padding to common component, to consider status bar on modal sheet if not consumed.
• Ignore consumed paths that are irrelevant, so undo perform is consistent with drew paths.
• Set title color accordingly to theme.

Attachments (Optional)

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[yamilmedina]

Black screen issue fix

[yamilmedina]

Several undo actions to execute fix.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 43.85%. Comparing base (37a0fe5) to head (4264455).

❗ Current head 4264455 differs from pull request most recent head e1629eb. Consider uploading reports for the commit e1629eb to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2919      +/-   ##
===========================================
- Coverage    43.95%   43.85%   -0.10%     
===========================================
  Files          441      441              
  Lines        14318    14317       -1     
  Branches      2478     2477       -1     
===========================================
- Hits          6293     6279      -14     
- Misses        7334     7354      +20     
+ Partials       691      684       -7     

Files	Coverage Δ
...m/wire/android/feature/sketch/DrawingToolPicker.kt	0.00% <ø> (ø)
...e/android/feature/sketch/DrawingCanvasComponent.kt	0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37a0fe5...e1629eb. Read the comment docs.

[github-actions]

Test Results

900 tests  ±0   900 ✅ ±0   12m 55s ⏱️ -29s
122 suites ±0     0 💤 ±0 
122 files   ±0     0 ❌ ±0 

Results for commit e1629eb. ± Comparison against base commit 37a0fe5.

♻️ This comment has been updated with latest results.

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[AndroidBob]

Build 4367 succeeded.

The build produced the following APK's:

• com.wire.android-v4.7.0(9482088)(9482088)-staging-compat.apk