View File allows editing of default text or .bmp files

VERIFIED INVALID

Status

CCK
CCK-Wizard
VERIFIED INVALID
17 years ago
17 years ago

People

(Reporter: bobj, Assigned: Shruti Vasudevan)

Tracking

Details

(Reporter)

Description

17 years ago
If you click on a "View File" button for any text file, it opens the
default file with Notepad and allows editing of the default file.

We should open the file read-only, so the user cannot accidentally
edit the default file.
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
(Reporter)

Comment 1

17 years ago
This can be seen in the dialog "Customizing the Browser - Part Two" when
you View File either readme.txt or panels.rdf.

Also, for .bmp the same problem arises because it opens the .bmp with Paint
allowing the user the modify the default .bmp.

See the dialog "Customize the CD Autorun Screen".  It allows editing of
the default background .bmp and the install.txt files.
Summary: View File allows editing of default text files → View File allows editing of default text or .bmp files
(Reporter)

Comment 2

17 years ago
Maybe aneasy fix is to change the file permissions to readonly.

But when we copy these files for a new config, we would have to make sure we
make them writable.

Updated

17 years ago
Whiteboard: nscckrtm?
(Reporter)

Comment 3

17 years ago
Is another option to use quikview.exe instead of notepad.exe?
Is it standard on all versions of Windows?

quikview does not allow you to edit files, but it does have a toolbar button
to open notepad or editing.  So it'd be an improvement, but still a user
could edit the file with another click.

quikview can also view .bmp files:

http://lxr.mozilla.org/mozilla/source/cck/cckwiz/iniFiles/Branding_page2.ini#113
could be changed from
  113 onCommand=OpenViewer(%ShellBgBitmap%)
to
  113 onCommand=command(%NCIQuikView% "%ShellBgBitmap")

The ugly part on my Win95, is the quikview opens in a small narrow window,
so I'm forced to resize the window to actually see the .txt or .bmp file.
I don't know how to change the default window size.
(Assignee)

Comment 4

17 years ago
Removing nscckrtm. Will fix this only if time permits.
Whiteboard: nscckrtm?

Updated

17 years ago
Keywords: relnote
(Assignee)

Updated

17 years ago
Target Milestone: --- → Future

Comment 5

17 years ago
Updating QA contact to me for half of the CCK bugs while bom-shik is gone.
QA Contact: blee → barrettl

Comment 6

17 years ago
qa to blee
QA Contact: barrettl → blee
(Assignee)

Updated

17 years ago
Depends on: 101439
Keywords: relnote
(Assignee)

Updated

17 years ago
Target Milestone: Future → ---

Comment 7

17 years ago
Information included in General section of CCK 6.2 Release Notes:

http://home.netscape.com/eng/mozilla/ns62/relnotes/cck.html
(Assignee)

Comment 8

17 years ago
Since bug 101439 has been fixed, this is no longer valid.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → INVALID

Comment 9

17 years ago
Marking Verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.