Opened 7 years ago

Closed 3 years ago

#1373 closed defect (fixed)

SusiMail doesn't handle multiple tabs well at all

Reported by: somewon Owned by: zzz
Priority: minor Milestone: 0.9.33
Component: apps/susimail Version:
Keywords: usability Cc:
Parent Tickets: Sensitive: no


Was trying to send an email through SusiMail?. Got my message pasted into the message box, but needed to grab a recipient's address from my inbox. Opened a second tab, navigated to Inbox, got the address on clipboard, etc. etc. - when I hit "Send" the page just instantly reverted to the last state of the other tab. It seems the message wasn't sent. No error is displayed.

I then reproduced this an additional time, after closing one of the tabs, reloading my inbox page, and attempting the same thing again.

I finally had to paste the recipient address elsewhere and fully restart my Susi session.

I would like it if SusiMail? handled multiple tabs well.


Change History (7)

comment:1 Changed 7 years ago by zzz

Yes this is a problem. There's internal state about where it thinks it is.

It used to be really bad. In 0.9.13 there's a ton of fixes for the worst cases, where the buttons wouldn't be right, and the back button would really mess things up.

But there's still one URL for most of the pages and one form. Need different URLs for the different pages so you know where the form is coming from and can bookmark pages to get back there.

comment:2 Changed 7 years ago by str4d

Keywords: usability added
Milestone: 0.9.16

comment:3 Changed 4 years ago by zzz

Milestone: 0.9.33
Owner: set to zzz
Status: newaccepted

Use UIDLs instead of list item number to load or delete emails, in 0d5b4dad7a85dae5fd2f5cda269c2eec41cbf169 to be 0.9.32-14
more to follow

comment:4 Changed 4 years ago by zzz

More in 28c71f37b9ad34ad212b853d731b60502ac6802c 0.9.32-14
more to follow

comment:5 Changed 4 years ago by zzz

Switch to using P-R-G, use page params, rework sort params, add config param, in several revs to be 0.9.32-16
more to follow

comment:6 Changed 4 years ago by zzz

Big refactor to remove state from session object, in 4ba8a3f0a555754bafe3629eb6bdd94b79726bfe 0.9.32-16.
This fixes OP issue.
More to follow.

comment:7 Changed 3 years ago by zzz

Resolution: fixed
Status: acceptedclosed

Most of the OP issues are fixed in 0.9.32-14, -16, and following fixes, and released in 0.9.33. There's a few minor fixes in 0.9.33-2 through 0.9.33-5 that will be in the 0.9.34 release.

What remains is the storage of a draft email in the session object. There's no folder support and so no other place to store a draft, and no way to support multiple draft emails. This is discussed in #2087 comment 6. It probably won't be addressed until 0.9.35.

Closing this ticket, watch #2087 for further updates.

Note: See TracTickets for help on using tickets.