SMS App Should Request Enough Messages to Fill the Initial Window
Closed, ResolvedPublic

Description

Currently, the SMS app requests 10 message when a conversation is opened. If that is not enough to fill the screen, the view looks strange and is difficult to scroll because the scroll bar does not show. The model should request enough messages to fill the view rather than a hard-coded number.

One way to do this is to literally request a single message, wait for the Dbus interface to reply (should be very quick in all cases), then see if the view is full. If not, request another message and repeat.

The solution to this will likely be relevant to solving T10186

sredman created this task.Dec 13 2018, 9:31 PM
sredman triaged this task as Normal priority.

Maybe we could just increase the hardcoded number as a first workaround? I think 20 should be enough for most cases

Maybe we could just increase the hardcoded number as a first workaround? I think 20 should be enough for most cases

Yeah, we could

But then there's going to be *that one guy* who has a 4K TV and wants to view his entire message history at once. Let's give the correct solution a try first - it hopefully won't be that hard - then we can try other solutions.

If you want to change the hard-coding so it's easier for you to use until I get time to work on the proper solution, go for it :)

sredman closed this task as Resolved.Jul 21 2019, 11:34 AM

Now we request a bunch, then check if there are enough and request more until there is