Closed
Bug 145243
Opened 22 years ago
Closed 21 years ago
warning should be presented if user attempts cancellation of publish
Categories
(SeaMonkey :: Composer, defect)
SeaMonkey
Composer
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.4beta
People
(Reporter: Brade, Assigned: jag+mozilla)
Details
(Whiteboard: [adt2] publish)
Attachments
(1 file)
2.50 KB,
patch
|
KaiE
:
review+
hewitt
:
superreview+
|
Details | Diff | Splinter Review |
if the user presses cancel in the publish progress dialog, we should warn them that they may be corrupting one or more of their files. Of course the string should say something more delicate like "interrupting the publish process while it is already in progress may cause incomplete files to be written to the server. Would you like to Continue or Cancel?" Robin--please help with wording; this is a very important bug to fix.
Suggested text: "Cancelling while publishing is in progress may result in your file(s) being incompletely transferred. Would you like to Continue or Cancel?"
Comment 2•22 years ago
|
||
This issue is much less important after current improvements in publishing code are accepted: Kathy's "serialize" changes so only one file is being transerred at a time, and with my current improvements to error reporting.
Comment 3•22 years ago
|
||
We will address this later
Updated•22 years ago
|
Reporter | ||
Updated•22 years ago
|
Assignee | ||
Updated•22 years ago
|
Target Milestone: mozilla1.2beta → mozilla1.4beta
Assignee | ||
Comment 6•21 years ago
|
||
Comment 7•21 years ago
|
||
Comment on attachment 120935 [details] [diff] [review] Notify user of risk of cancelling publishing sr=me
Attachment #120935 -
Flags: superreview+
Attachment #120935 -
Flags: review?
Comment 8•21 years ago
|
||
Comment on attachment 120935 [details] [diff] [review] Notify user of risk of cancelling publishing Looks reasonable to me, r=kaie But note I'm not the module owner. Kathy, do you think we need another review?
Attachment #120935 -
Flags: review? → review+
Reporter | ||
Comment 9•21 years ago
|
||
moa=brade
Assignee | ||
Comment 10•21 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 11•21 years ago
|
||
This is fixed on the macho trunk (2003-06-04-08) and branch (2003-06-04-05) builds. Need to check win32 build.
Comment 12•21 years ago
|
||
Fixed on the win32 trunk (2003-06-03-08) and branch (2003-06-04-05) builds
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•