Qksms: Copy partial text

[moezbhatti]

Instead of needing to copy an entire message, you should be able to just copy the part that you want

[GeoZac]

android:textIsSelectable attribute to message body view should do the trick but those text view seems to forward their touch event to the root layout they are in

[moezbhatti]

Unfortunately it's not quite so simple, because long-pressing the message is currently how you access multi-select. So all of that will need to be reworked

[GeoZac]

Since the root view already has the OnLongClickListener , with forwardTouches(view) disabled, the multiselect works when touch is anywhere outside the message body( with the root view), which is an appreciable area.

_P.S.:I had been playing around with these changes for last few days, and no regressions were observed_

[moezbhatti]

Hmm interesting, I hadn't thought about leaving the hold-for-multiselect. Although I think it might be a bit confusing if you have to hold outside of the message in order to select it

I was thinking of implementing #1277 at the same time as this, as a solution to that problem

[moezbhatti]

Finally working on this! Tracking progress in https://github.com/moezbhatti/qksms/tree/partial-copy

[GeoZac]

Hats off the the amazing work. Really like the new implementation, but I cam across an issue where a third tab in the meaasge body ( first one opens the options, second dismissses it) doesn't show the options, but I can select the text, is this intentional ?

[moezbhatti]

That's on my to-do list, this is still a WIP!

[moezbhatti]

I'll let you know when it's closer to being ready - probably later this week