limit convo bubble text to 5000 chars#389
Merged
octoshrimpy merged 1 commit intoquik-sms:masterfrom Mar 28, 2025
gavine99:truncate-long-message-text-236
Merged
limit convo bubble text to 5000 chars#389octoshrimpy merged 1 commit intoquik-sms:masterfrom gavine99:truncate-long-message-text-236
octoshrimpy merged 1 commit intoquik-sms:masterfrom
gavine99:truncate-long-message-text-236
Conversation
Collaborator
|
@gavine99 i'm a dummy who didn't merge things in order, could you fix the conflicts? |
add instruction status message when bubble char limit exceeded. add share text feature. make copy and share text menu options only available when there is text in selected messages. new FileProvider to share messages text files out of app's cache dir. add message subject to copy messages text feature. a lot of refactoring of MessagesAdapter class. add new FileUtils class. convert file logging to use new FileUtils class. change MediaRecorder to use new FileUtils class. housekeeping worker now cleans messages text cache files. housekeeping worker now checks cache files older than 2 hours before deleting.
Contributor
Author
|
rebased this branch onto latest master and it should be ok to merge now |
Collaborator
|
thank you! you should join the matrix chat. |
Closed
This was referenced Nov 13, 2025
Merged
Closed
Merged
This was referenced Dec 9, 2025
Closed
Merged
Merged
Merged
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
lots of point changes that flowed from limiting and sharing text feature;
limit bubble display of text to 5000 chars with ellipses.
add instruction status message when bubble char limit exceeded. add share text feature.
make copy and share text menu options only available when there is text in selected messages.
new FileProvider to share messages text files out of app's cache dir.
add message subject to copy messages text feature.
a lot of refactoring of MessagesAdapter class.
add new FileUtils class.
convert file logging to use new FileUtils class.
convert MediaRecorder to use new FileUtils class.
housekeeping worker now cleans messages text cache files.
housekeeping worker now checks cache files older than 2 hours before deleting.
screen-record.mp4
closes #236