yetangitu/deltachat
1
0
Fork 0
mirror of https://github.com/deltachat/deltachat-android.git synced 2025-10-03 01:39:18 +02:00
Code Issues Releases Wiki Activity

drop disappearing messages options 1 and 30 minutes

Browse source 
- with the recent change to start the timer when the chat is opened,
  one minute is too easy too short to miss important information

- "30 minutes" does not really fit into the distribution;
  all other option have at least the factor 5 to the previous one,
  from "30 minutes" to "60 minutes", the time is doubled only.

removing the overall number of options and dropping the ones without larger usecases,
does not only simplifies UI;
having less tiny details also reduces noise in groups
where ppl people changing it very often for various weird reasons
(found the latter point somewhere in the Signal forum, btw)
This commit is contained in:
B. Petersen 2025-08-22 15:43:22 +02:00 • committed by bjoern
parent d4ddb68137
commit 5d7c701da8
2 changed files with 12 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

14
src/main/java/org/thoughtcrime/securesms/EphemeralMessagesDialog.java
View file

@ -60,14 +60,12 @@ public class EphemeralMessagesDialog {
.setPositiveButton(R.string.ok, (dialog, which) -> { .setPositiveButton(R.string.ok, (dialog, which) -> {
final long burnAfter; final long burnAfter;
switch (selectedChoice[0]) { switch (selectedChoice[0]) {
case 1: burnAfter = TimeUnit.MINUTES.toSeconds(1); break; case 1: burnAfter = TimeUnit.MINUTES.toSeconds(5); break;
case 2: burnAfter = TimeUnit.MINUTES.toSeconds(5); break; case 2: burnAfter = TimeUnit.HOURS.toSeconds(1); break;
case 3: burnAfter = TimeUnit.MINUTES.toSeconds(30); break; case 3: burnAfter = TimeUnit.DAYS.toSeconds(1); break;
case 4: burnAfter = TimeUnit.HOURS.toSeconds(1); break; case 4: burnAfter = TimeUnit.DAYS.toSeconds(7); break;
case 5: burnAfter = TimeUnit.DAYS.toSeconds(1); break; case 5: burnAfter = TimeUnit.DAYS.toSeconds(35); break;
case 6: burnAfter = TimeUnit.DAYS.toSeconds(7); break; case 6: burnAfter = TimeUnit.DAYS.toSeconds(365); break;
case 7: burnAfter = TimeUnit.DAYS.toSeconds(35); break;
case 8: burnAfter = TimeUnit.DAYS.toSeconds(365); break;
default: burnAfter = 0; break; default: burnAfter = 0; break;
} }
listener.onTimeSelected(burnAfter); listener.onTimeSelected(burnAfter);

14
src/main/res/values/arrays.xml
View file

@ -49,14 +49,12 @@
<string-array name="ephemeral_message_durations"> <string-array name="ephemeral_message_durations">
<!-- 0 --> <item>@string/off</item> <!-- 0 --> <item>@string/off</item>
<!-- 1 --> <item>@string/after_1_minute</item> <!-- 1 --> <item>@string/after_5_minutes</item>
<!-- 2 --> <item>@string/after_5_minutes</item> <!-- 2 --> <item>@string/autodel_after_1_hour</item>
<!-- 3 --> <item>@string/after_30_minutes</item> <!-- 3 --> <item>@string/autodel_after_1_day</item>
<!-- 4 --> <item>@string/autodel_after_1_hour</item> <!-- 4 --> <item>@string/autodel_after_1_week</item>
<!-- 5 --> <item>@string/autodel_after_1_day</item> <!-- 5 --> <item>@string/after_5_weeks</item>
<!-- 6 --> <item>@string/autodel_after_1_week</item> <!-- 6 --> <item>@string/autodel_after_1_year</item>
<!-- 7 --> <item>@string/after_5_weeks</item>
<!-- 8 --> <item>@string/autodel_after_1_year</item>
</string-array> </string-array>
<string-array name="mute_durations"> <string-array name="mute_durations">