Switch form extending AlertDialog to just Dialog
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
Research direction
Start with sentry-android-core/src/main/java/io/sentry/android/core/SentryUserFeedbackForm.java, especially the dialog setup around the linked line. Confirm that the form does not rely on AlertDialog behavior, switch its inheritance to Dialog, and verify that the user feedback form still displays its own content correctly.
Written by the indexing model from the issue text.
Description
Description
We're not using anything from AlertDialog and throwing away its content by setting our own, so there's no reason to extend from AlertDialog.
- Dominant language
- Kotlin
- Stars
- 1.4k
- Forks
- 478
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 70
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from getsentry/sentry-java
-
Improvement Java Platform: Java
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
getsentry/sentry-java#6145 · 1 comment ·
-
Bug Java Platform: Java
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
getsentry/sentry-java#6138 · 1 comment ·
-
Feature Java Platform: Java Spans
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
getsentry/sentry-java#5984 · 1 comment ·
-
Android Task Traces
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
getsentry/sentry-java#5376 · 1 comment ·
-
Android Docs Errors
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
getsentry/sentry-java#5375 · 1 comment ·
All issues in getsentry/sentry-java
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
2 rename bugs Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
ankidroid/Anki-Android#21999 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
openwallet-foundation/multipaz#2028 ·
-
Doc: Oppdater README Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100