Phone icon indicating copy to clipboard operation
Phone copied to clipboard

WIP: Initial support for sim colors

Open derrohrbach opened this issue 2 years ago • 4 comments

Here is my initial draft for sim color support. Feel free to already comment on it. Things left to do:

  • [ ] Move sim color conversions to commons library, so we can use it in SMS app too
  • [x] Test with more designs
  • [x] Check/Work on CallActivity and ConferenceActivity
  • Maybe also rework SelectSIMDialog (most likely seperate PR)

This fixes/implements: https://github.com/SimpleMobileTools/Simple-Dialer/issues/322

derrohrbach avatar Jan 17 '24 01:01 derrohrbach

@derrohrbach I'll check this soon...

If the PR isn't ready to merge yet, you can convert it to a draft PR to signal that.

naveensingh avatar Jan 20 '24 15:01 naveensingh

@naveensingh Hey, thanks. I did not know that, since I'm used to the GitLab workflow where you just name your MR "WIP".

derrohrbach avatar Jan 22 '24 21:01 derrohrbach

Only thing missing is moving the extension function to the common library. What is the workflow for that? Do I first submit a PR there and wait for a version to be built/published so i can update the reference in this project?

Or should I just copy this one function over to the SMS app?

derrohrbach avatar Jan 22 '24 21:01 derrohrbach

@derrohrbach

What is the workflow for that? Do I first submit a PR there and wait for a version to be built/published so i can update the reference in this project?

Please raise a PR in Commons. Once the PR is merged, anyone can trigger a Jitpack build at this page: https://jitpack.io/#org.fossify/commons/

naveensingh avatar Mar 17 '24 02:03 naveensingh