Skip to content

Propose: Remove "Lisp Edit" sub-item from FILEBROWSER Edit menu #2384

@MattHeffron

Description

@MattHeffron

I propose removing the "Lisp Edit" sub-item from the "Edit" menu entry of FILEBROWSER.

Currently, what this does, is to, check if it is LISPSOURCEFILEP, if so, it checks if it is already loaded and the current version (by comparing file paths). If it isn't, then it asks to load the file PROP.
If it is already loaded and current, or it just did the LOAD PROP, it opens the editor on the fileCOMS. (I.e., it does (ED rootOfFilename '(FILES :DONTWAIT))

There may be some cases where this is useful, but very few at most. And there are ways that the implementation of this process can fail its apparent intent.

In addition to removing the "Lisp Edit" sub-item, (which makes the "Tedit" sub-item redundant, so remove that also), I suggest in the Tedit case, perform a LISPSOURCEFILEP check and prompt for user confirmation with a message like: "Directly editing a Medley lisp source file is not recommended. Are you sure?"

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions