sredman (Simon Redman)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
May 13 2017, 6:47 AM (92 w, 5 d)
Availability
Available

Recent Activity

Yesterday

sredman added a comment to D18212: Use Storage Access Framework on SDK >= 21 (Lollipop and above).

Cool! I like the warning popup!

Wed, Feb 20, 11:21 PM · KDE Connect

Sat, Feb 9

sredman accepted D17292: [SMS App] Allow passing a message via args.

Reminder that there is a spelling error on line 37 of ConversationList.qml

Sat, Feb 9, 4:27 PM · KDE Connect

Thu, Jan 24

sredman added a comment to D16071: Use input field from Kaidan.

I don't have a problem with this. Like you say, if the future gives us what we need, we can change then. Of course, this patch will take a little bit of effort to rebase since lots of things around the text field have changed :/

Thu, Jan 24, 5:47 PM · KDE Connect
sredman committed R224:3b92b0de1d57: Add dependency on kirigami to SMS app (authored by sredman).
Add dependency on kirigami to SMS app
Thu, Jan 24, 5:41 PM
sredman closed D17572: Add dependency on kirigami to SMS app.
Thu, Jan 24, 5:40 PM · KDE Connect

Wed, Jan 23

sredman committed R224:a14b39d54116: [SMS App] Make sms app not crash when conversation is selected with no devices… (authored by sredman).
[SMS App] Make sms app not crash when conversation is selected with no devices…
Wed, Jan 23, 1:58 AM
sredman closed T10184: SMS App Should Gracefully Handle Phone Disconnection, a subtask of T8337: SMS App Initial Release, as Resolved.
Wed, Jan 23, 1:58 AM · KDE Connect
sredman closed D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Wed, Jan 23, 1:58 AM · KDE Connect
sredman closed T10184: SMS App Should Gracefully Handle Phone Disconnection as Resolved by committing R224:a14b39d54116: [SMS App] Make sms app not crash when conversation is selected with no devices….
Wed, Jan 23, 1:58 AM · KDE Connect
sredman planned changes to D17616: [SMS App] Improve "No devices available" warning.

I just noticed today that the tooltip disappears after a few seconds. I will have to find a solution to this problem.

Wed, Jan 23, 1:28 AM · KDE Connect

Jan 21 2019

sredman committed R224:74ba660cad19: Cache and delete old ConversationsDbusInterfaces to avoid memory leak (authored by sredman).
Cache and delete old ConversationsDbusInterfaces to avoid memory leak
Jan 21 2019, 7:07 PM
sredman closed D16553: Cache and delete old ConversationsDbusInterfaces to avoid memory leak.
Jan 21 2019, 7:01 PM · KDE Connect
sredman added a comment to D18140: Add plugin for taking photos on remote device.
In D18140#392884, @apol wrote:

Can you elaborate a bit on the use-case?

I also find the use case a bit obscure.

Unless you are a spy and want to take photos from your phone without anyone noticing, I see no use of this plugin.

Jan 21 2019, 5:56 PM · KDE Connect

Jan 18 2019

sredman added a comment to D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.

I can confirm that it doesn't crash when I unpair my device, but now there's the "No device" warning and the conversation list visible simultaneously

Jan 18 2019, 5:51 PM · KDE Connect

Jan 14 2019

sredman updated the diff for D16553: Cache and delete old ConversationsDbusInterfaces to avoid memory leak.
  • Clairify pointer-saving comment
Jan 14 2019, 8:42 PM · KDE Connect

Jan 13 2019

sredman added a comment to D18212: Use Storage Access Framework on SDK >= 21 (Lollipop and above).

The downside of this patch is it changes the default: I was very confused that I couldn't immediately see my USB drive and internal storage from Dolphin. Our per-plugin settings are very non-obvious :/. Is it possible to either display a dummy "you need to configure settings" file, or to show everything by default (as things currently are) and leave the settings so advanced users can hide filesystems if they like?

Jan 13 2019, 4:26 PM · KDE Connect
sredman added a comment to D18212: Use Storage Access Framework on SDK >= 21 (Lollipop and above).

USB OTG Testing looks good:

  • Before this patch, I was able to read/write USB OTG storage already
  • With this patch, I was able to add USB OTG as an exposed filesystem
  • After adding as an exposed filesystem, I was able to read/write the USB OTG drive
Jan 13 2019, 4:24 PM · KDE Connect

Jan 2 2019

sredman closed T8288: Drag n Drop Filetransfer on the plasmoid, a subtask of T8293: Improve file sharing, as Resolved.
Jan 2 2019, 5:28 PM · Junior Jobs, KDE Connect
sredman closed T8288: Drag n Drop Filetransfer on the plasmoid as Resolved.
Jan 2 2019, 5:28 PM · Junior Jobs, KDE Connect
sredman committed R224:bafbc344b931: Implement Drag n Drop file transfer on the plasmoid (authored by sredman).
Implement Drag n Drop file transfer on the plasmoid
Jan 2 2019, 5:26 PM
sredman added a commit to T8288: Drag n Drop Filetransfer on the plasmoid: R224:bafbc344b931: Implement Drag n Drop file transfer on the plasmoid.
Jan 2 2019, 5:26 PM · Junior Jobs, KDE Connect
sredman closed D17611: Implement Drag n Drop file transfer on the plasmoid.
Jan 2 2019, 5:26 PM · KDE Connect

Dec 26 2018

sredman added a comment to D17790: Port away from QCA.

Using the command line tool means this is a runtime dependency only. You could run kdeconnect without having openssl installed and it would fail mysteriously.

We can either check that it exists and output a specific error, or change to a compile-time dependency (libopenssl? Not sure if it exists).

Dec 26 2018, 9:48 PM · KDE Connect

Dec 23 2018

sredman accepted D17611: Implement Drag n Drop file transfer on the plasmoid.

Hello,

I have tested a few things but could not manage to get the tooltip to show everywhere without causing issues with the rest of the widgets. Putting the tooltip on top (with the property z greater than 0) prevents underlying widgets from receiving mouse movements, thus making them look weird. I do not have any improvement for now before merging.

Dec 23 2018, 11:49 PM · KDE Connect

Dec 21 2018

sredman renamed T10208: SMS App should have click-able URLs from Make URLs click-able to SMS App should have click-able URLs.
Dec 21 2018, 3:38 PM · KDE Connect
sredman created T10208: SMS App should have click-able URLs.
Dec 21 2018, 3:29 PM · KDE Connect

Dec 18 2018

sredman added a comment to D17292: [SMS App] Allow passing a message via args.
  • Only fill in message field and make it cancellable
Dec 18 2018, 6:49 AM · KDE Connect
sredman renamed T10186: SMS App Should Instantly Show the Most-Recent Message when Conversation is Opened from SMS App Should Show the Most-Recent Message when Conversation is Opened to SMS App Should Instantly Show the Most-Recent Message when Conversation is Opened.
Dec 18 2018, 6:27 AM · KDE Connect
sredman updated the diff for D17616: [SMS App] Improve "No devices available" warning.
  • Change button for MouseArea
Dec 18 2018, 6:11 AM · KDE Connect
sredman added a comment to D17611: Implement Drag n Drop file transfer on the plasmoid.

Very cool! I can now drop onto the desktop plasmoid and have it share without the weird popup, and I can expand the toolbar plasmoid by hovering over it while dragging. Nice!

Dec 18 2018, 5:35 AM · KDE Connect
sredman planned changes to D17553: Use typedef threadID_t qint64 instead of hard-coding.

This revision cannot currently land: It needs either updated Qt Tools support (https://bugreports.qt.io/browse/QTBUG-72522) or we need to somehow upgrade our build system to allow custom types to be defined in the dbus XML

Dec 18 2018, 2:03 AM · KDE Connect

Dec 17 2018

sredman updated the diff for D17612: [SMS App] Add highlighting to ConversationList view.

Only do patch since D17634

Dec 17 2018, 7:25 PM · KDE Connect
sredman added a dependency for D17612: [SMS App] Add highlighting to ConversationList view: D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Dec 17 2018, 7:24 PM · KDE Connect
sredman added a dependent revision for D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected: D17612: [SMS App] Add highlighting to ConversationList view.
Dec 17 2018, 7:24 PM · KDE Connect
sredman updated the summary of D17612: [SMS App] Add highlighting to ConversationList view.
Dec 17 2018, 7:24 PM · KDE Connect
sredman updated the diff for D17612: [SMS App] Add highlighting to ConversationList view.
  • Keep currently-open chat highlighted even when the list loses focus
Dec 17 2018, 7:23 PM · KDE Connect
sredman committed R224:5b213e614498: [SMS App] Improve filter (authored by sredman).
[SMS App] Improve filter
Dec 17 2018, 4:28 PM
sredman closed T8341: Contacts search box does not work, a subtask of T8337: SMS App Initial Release, as Resolved.
Dec 17 2018, 4:28 PM · KDE Connect
sredman closed T8341: Contacts search box does not work as Resolved by committing R224:5b213e614498: [SMS App] Improve filter.
Dec 17 2018, 4:28 PM · KDE Connect
sredman closed D17614: [SMS App] Improve filter.
Dec 17 2018, 4:26 PM · KDE Connect

Dec 16 2018

sredman removed a revision from T10183: SMS App should display some loading indicator while loading: D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Dec 16 2018, 11:40 PM · KDE Connect
sredman removed a task from D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected: T10183: SMS App should display some loading indicator while loading.
Dec 16 2018, 11:40 PM · KDE Connect
sredman added a task to D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected: T10184: SMS App Should Gracefully Handle Phone Disconnection.
Dec 16 2018, 11:39 PM · KDE Connect
sredman added a revision to T10184: SMS App Should Gracefully Handle Phone Disconnection: D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Dec 16 2018, 11:39 PM · KDE Connect
sredman requested review of D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Dec 16 2018, 8:22 PM · KDE Connect
sredman added a revision to T10183: SMS App should display some loading indicator while loading: D17634: [SMS App] Make sms app not crash when conversation is selected with no devices connected.
Dec 16 2018, 8:22 PM · KDE Connect
sredman updated the diff for D16553: Cache and delete old ConversationsDbusInterfaces to avoid memory leak.

Rebase onto master to avoid merge conflict

Dec 16 2018, 5:05 AM · KDE Connect
sredman updated the summary of D17616: [SMS App] Improve "No devices available" warning.
Dec 16 2018, 2:01 AM · KDE Connect
sredman requested review of D17616: [SMS App] Improve "No devices available" warning.
Dec 16 2018, 2:00 AM · KDE Connect
sredman committed R224:657dede4bf28: Add contributors to smsapp QML files (authored by sredman).
Add contributors to smsapp QML files
Dec 16 2018, 12:26 AM
sredman added a comment to D17611: Implement Drag n Drop file transfer on the plasmoid.

This is cool! But kind of finicky:

Dec 16 2018, 12:23 AM · KDE Connect
sredman requested review of D17614: [SMS App] Improve filter.
Dec 16 2018, 12:10 AM · KDE Connect
sredman added a revision to T8341: Contacts search box does not work: D17614: [SMS App] Improve filter.
Dec 16 2018, 12:10 AM · KDE Connect

Dec 15 2018

sredman requested review of D17612: [SMS App] Add highlighting to ConversationList view.
Dec 15 2018, 11:55 PM · KDE Connect

Dec 14 2018

sredman accepted D17250: Fixed Header in 3 plugin pages: Run Command, Multimedia control, Remote input.
Dec 14 2018, 10:09 PM · KDE Connect
sredman committed R224:900943995660: Handle case where more messages were requested but there were no more available… (authored by sredman).
Handle case where more messages were requested but there were no more available…
Dec 14 2018, 5:03 PM

Dec 13 2018

sredman added a comment to T10187: SMS App Should Request Enough Messages to Fill the Initial Window.

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

Dec 13 2018, 11:03 PM · KDE Connect
sredman raised the priority of T10184: SMS App Should Gracefully Handle Phone Disconnection from Normal to High.
Dec 13 2018, 9:34 PM · KDE Connect
sredman renamed T10185: SMS App Should Gracefully Handle Phone Re-Connection from Messaging App Should Gracefully Handle Phone Re-Connection to SMS App Should Gracefully Handle Phone Re-Connection.
Dec 13 2018, 9:33 PM · KDE Connect
sredman renamed T10184: SMS App Should Gracefully Handle Phone Disconnection from Messaging App Should Gracefully Handle Phone Disconnection to SMS App Should Gracefully Handle Phone Disconnection.
Dec 13 2018, 9:33 PM · KDE Connect
sredman renamed T8337: SMS App Initial Release from SMS App to SMS App Initial Release.
Dec 13 2018, 9:32 PM · KDE Connect
sredman triaged T10187: SMS App Should Request Enough Messages to Fill the Initial Window as Normal priority.
Dec 13 2018, 9:31 PM · KDE Connect
sredman triaged T10186: SMS App Should Instantly Show the Most-Recent Message when Conversation is Opened as High priority.
Dec 13 2018, 9:28 PM · KDE Connect
sredman added a comment to T10184: SMS App Should Gracefully Handle Phone Disconnection.

This is essentially a re-statement of bug 401587

Dec 13 2018, 9:12 PM · KDE Connect
sredman created T10185: SMS App Should Gracefully Handle Phone Re-Connection.
Dec 13 2018, 9:11 PM · KDE Connect
sredman triaged T10184: SMS App Should Gracefully Handle Phone Disconnection as Normal priority.
Dec 13 2018, 9:06 PM · KDE Connect
sredman triaged T10183: SMS App should display some loading indicator while loading as Normal priority.
Dec 13 2018, 9:01 PM · KDE Connect
sredman closed T9556: Should be able to view older messages by scrolling up as Resolved.

Resolved by D15979

Dec 13 2018, 8:56 PM · KDE Connect
sredman closed T9556: Should be able to view older messages by scrolling up, a subtask of T8337: SMS App Initial Release, as Resolved.
Dec 13 2018, 8:56 PM · KDE Connect
sredman updated subscribers of D17572: Add dependency on kirigami to SMS app.
Dec 13 2018, 8:53 PM · KDE Connect
sredman updated the summary of D17572: Add dependency on kirigami to SMS app.
Dec 13 2018, 8:51 PM · KDE Connect
sredman requested review of D17572: Add dependency on kirigami to SMS app.
Dec 13 2018, 8:50 PM · KDE Connect
sredman requested review of D17553: Use typedef threadID_t qint64 instead of hard-coding.
Dec 13 2018, 6:37 AM · KDE Connect
sredman committed R224:2c35c1af518d: [SMS App] Allow scrolling up to load and display older messages (authored by sredman).
[SMS App] Allow scrolling up to load and display older messages
Dec 13 2018, 5:50 AM
sredman closed D15979: [SMS App] Allow scrolling up to load and display older messages.
Dec 13 2018, 5:50 AM · KDE Connect
sredman committed R224:7025fd67959a: Un-break build. Apparently I didn't test as I thought I had. (authored by sredman).
Un-break build. Apparently I didn't test as I thought I had.
Dec 13 2018, 5:49 AM
sredman updated the diff for D15979: [SMS App] Allow scrolling up to load and display older messages.
  • Re-Re-Rebase onto Master including D16475
Dec 13 2018, 5:45 AM · KDE Connect
sredman closed D16475: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching.
Dec 13 2018, 5:43 AM · KDE Connect
sredman committed R224:019e8d6a1c0e: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching (authored by sredman).
[SMS App] Make requestMoreMessages asynchronous, blocking, and caching
Dec 13 2018, 5:43 AM
sredman updated the diff for D16475: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching.
  • Clean inequalities
Dec 13 2018, 5:40 AM · KDE Connect
sredman committed R225:5eb368c630cf: Add incremental notifications plugin settings database upgrade (authored by sredman).
Add incremental notifications plugin settings database upgrade
Dec 13 2018, 2:12 AM
sredman closed D17521: Add incremental notifications plugin settings database upgrade.
Dec 13 2018, 2:12 AM · KDE Connect
sredman committed R225:6ba6842fc75e: Privacy Options for Notifications Forwarded to Desktop (authored by alexkovrigin).
Privacy Options for Notifications Forwarded to Desktop
Dec 13 2018, 2:11 AM
sredman closed D17126: Privacy Options for Notifications Forwarded to Desktop.
Dec 13 2018, 2:11 AM · KDE Connect

Dec 12 2018

sredman added a dependency for D17521: Add incremental notifications plugin settings database upgrade: D17126: Privacy Options for Notifications Forwarded to Desktop.
Dec 12 2018, 4:30 AM · KDE Connect
sredman added a dependent revision for D17126: Privacy Options for Notifications Forwarded to Desktop: D17521: Add incremental notifications plugin settings database upgrade.
Dec 12 2018, 4:30 AM · KDE Connect
sredman accepted D17126: Privacy Options for Notifications Forwarded to Desktop.

Sorry this took so long

Dec 12 2018, 4:30 AM · KDE Connect
sredman requested review of D17521: Add incremental notifications plugin settings database upgrade.
Dec 12 2018, 4:28 AM · KDE Connect
sredman added a comment to D17250: Fixed Header in 3 plugin pages: Run Command, Multimedia control, Remote input.

Sorry, this is going to feel like a tease. This is really good, and I'm sorry I'm so picky :)

Dec 12 2018, 1:21 AM · KDE Connect
sredman updated the diff for D16475: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching.
  • Rebase on master (again)
Dec 12 2018, 1:10 AM · KDE Connect
sredman committed R225:dda6a5104cf7: [Android] Change ThreadID to long (authored by sredman).
[Android] Change ThreadID to long
Dec 12 2018, 1:05 AM
sredman committed R224:5cdfb9b087e1: [Desktop] Change ThreadID to long (authored by sredman).
[Desktop] Change ThreadID to long
Dec 12 2018, 1:03 AM
sredman closed D17516: [Desktop] Change ThreadID to long.
Dec 12 2018, 1:03 AM · KDE Connect
sredman closed D17517: [Android] Change ThreadID to long.
Dec 12 2018, 1:03 AM · KDE Connect
sredman updated the diff for D15979: [SMS App] Allow scrolling up to load and display older messages.

Rebase on master via D16475

Dec 12 2018, 12:57 AM · KDE Connect
sredman updated the diff for D16475: [SMS App] Make requestMoreMessages asynchronous, blocking, and caching.

Rebase onto master

Dec 12 2018, 12:46 AM · KDE Connect
sredman updated the test plan for D17516: [Desktop] Change ThreadID to long.
Dec 12 2018, 12:37 AM · KDE Connect
sredman updated the test plan for D17517: [Android] Change ThreadID to long.
Dec 12 2018, 12:37 AM · KDE Connect
sredman requested review of D17517: [Android] Change ThreadID to long.
Dec 12 2018, 12:32 AM · KDE Connect
sredman requested review of D17516: [Desktop] Change ThreadID to long.
Dec 12 2018, 12:32 AM · KDE Connect