Closed Bug 17102 Opened 25 years ago Closed 25 years ago

[DOGFOOD][PP] Linux file picker UI needs 'Replace existing file?' dialog

Categories

(Core :: DOM: Editor, defect, P1)

Product:
Core
Component:
DOM: Editor
Platform:
Other
Linux
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: sujay, Assigned: waqar)

References

Details

(Whiteboard: [PDT+] will verify in 11/30 build.) 

using 10/22 build of apprunner on linux

1) launch apprunner
2) launch editor
3) insert image
4) save file to disk
5) open a new blank doc and make an edit
6) Save As file to same filename

notice error panel comes up...you can't save back to the original file name.

linux only.
Assignee: buster → sfraser
Priority: P3 → P1
Summary: [PP]can't save over existing file → [DOGFOOD][PP]can't save over existing file
Target Milestone: M12
setting to M12, bumping priority to P1, assigning to Simon and suggesting this
be a dogfood candidate
Whiteboard: [PDT+]
Putting on [PDT+]radar.
Blocks: 12658
Blocks: 17907
Assignee: sfraser → pavlov
Oops, I change the wrong bug. This belongs here now:

This is a file picker issue, not a composer issue. On Mac (and probably windows,
though I don't have a build to check on), the native UI puts up the dialog that
gets the user to confirm that they want to save over an existing file. That UI
needs to be implemenented on linux.

The actual file replacement does not happen until later, and that's covered in
bug 9570.
No longer blocks: 9570
Summary: [DOGFOOD][PP]can't save over existing file → [DOGFOOD][PP] Linux file picker UI needs 'Replace existing file?' dialog
Adjust summary. When this is done, please check that replacing the contents of an
exisiting file works ok (bug 9570).
Whiteboard: [PDT+] → [PDT+] [by 11/19]
This one has not been investigated and it may require assistance from akkana
and/or pavlov -- this is a linux only issue and is in the file widget code.
Whiteboard: [PDT+] [by 11/19] → [PDT+]
Whiteboard: [PDT+] → [PDT+] need to re-write linux file picker
the linux filepicker needs to  be rewritten.
rickg would like to offer the help of waqar to fix this.  ok?
Thanks!  Should we just reassign it to him and cc pavlov?
I'd like to get together with Waqar sometime (soon) and go over the file widget
stuff.  I have a few other bugs that should be factored in to the file widget.
*** Bug 9570 has been marked as a duplicate of this bug. ***
Assignee: pavlov → waqar
reassigning to waqar
Status: NEW → ASSIGNED
Whiteboard: [PDT+] need to re-write linux file picker → [PDT+] need to re-write linux file picker 11/23/99
I have fix for it. I need someone to code review.
I hope you are asking in for review in places other than the bug report!  Please
send diffs to pavlov and sdagley via email.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Fix checked and verified. Closing the bug. Reviewed by Chris Mcafee.
Whiteboard: [PDT+] need to re-write linux file picker 11/23/99 → [PDT+] will verify in 11/30 build.
Status: RESOLVED → VERIFIED
verified in 11/30 build.
No longer blocks: 17907
You need to log in before you can comment on or make changes to this bug.