-
-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update menus.scrbl #693
base: master
Are you sure you want to change the base?
Update menus.scrbl #693
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -543,3 +543,47 @@ brings the corresponding window to the front. | |
DrRacket to other languages.} | ||
|
||
] | ||
|
||
|
||
@; ---------------------------------------- | ||
|
||
@section{@onscreen{Context menu}} | ||
|
||
The following additional menu items may be available when activating the context menu in an | ||
editing panel. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We don't use the phrase "editing panel" either. For this one, I think that "either the definitions or interactions window" is probably the right phrase. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 😅 thanks! |
||
|
||
@itemize[ | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Some of these items appear in some context and some in others. How about breaking them up into lists based on a description of where they might appear? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Go for it, but I would not worry too much |
||
@item{@defmenuitem{Tack/Untack Arrow(s)} Tack binding relationship arrows so | ||
they do not disappear. Repeat to untack arrows.} | ||
|
||
@item{@defmenuitem{Open Defining File} Opens the file with the currently | ||
selected identifier was defined.} | ||
|
||
@item{@defmenuitem{Jump to Next Bound Occurence} Selects the next occurence of the variable.} | ||
|
||
@item{@defmenuitem{Jump to Binding Occurence} Selects where the identifier was bound in the text. | ||
If imported the name of the imported library is selected.} | ||
|
||
@item{@defmenuitem{Add Require Prefix} Adds prefix-in, prompts for prefix, and renames | ||
instances of imported identifier to include the prefix.} | ||
|
||
@item{@defmenuitem{Remove Unused Requires} Refactoring tool to remove unused | ||
requires in module.} | ||
|
||
@item{@defmenuitem{Rename} Renames all occurences of selected variable.} | ||
|
||
@item{@defmenuitem{Collapse S-expression} Caused the current S-expression to | ||
be renderied as (...)} | ||
|
||
@item{@defmenuitem{Collapse S-expression} Expands S-expression displayed as | ||
(...) back to origninal text.} | ||
|
||
@item{@defmenuitem{Search in Help Desk for ...} Opens browser and triggers | ||
search of documentation for selected term.} | ||
|
||
@item{@defmenuitem{Save Image...} Save currently selected image to file.} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it saves the one you click on, not the selected one. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. good point |
||
] | ||
|
||
Standard editng functions are also available via the context menu: Undo, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Typo here. |
||
Redo, copy, cut, paste, Select All, and Select All. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These should probably also be itemized. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we use the phrase "context menu" elsewhere in the documentation. In the
racket/gui
docs, it is called a "popup menu".I guess a good idea is to look around in the drracket docs and see what term is used and then either change them all or change this one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
popup is fine.
I might be tempted to add 'also known as 'context menu' to the racket gui docs - and add an example to the gallery https://docs.racket-lang.org/gui/Widget_Gallery.html