Skip to content
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

Add render-menu function. #3296

Merged
merged 8 commits into from
Dec 22, 2023
Merged

Add render-menu function. #3296

merged 8 commits into from
Dec 22, 2023

Conversation

jmercouris
Copy link
Member

Description

  • This adds a function render-menu. This function can be used to generate menus for internal modes.

It also a fixes an issue where editor buffers could not have execute Lisp URLs. It fixes this by classing urls starting with editor as internal URLs.

Fixes #3162

Checklist:

  • Git branch state is mergable.
  • Changelog is up to date (via a separate commit).
  • New dependencies are accounted for.
  • Documentation is up to date.
  • Compilation and tests ((asdf:test-system :nyxt/gi-gtk))
    • No new compilation warnings.
    • Tests are sufficient.

It not *only* lists commands of the pcakge, it ALSO lists commands within the
Nyxt package. Probably because this function is primarily in service of the
prompt-buffer execute-command function.
@jmercouris jmercouris requested a review from aadcg December 21, 2023 22:10
@@ -441,8 +441,7 @@ The `input' should be a valid Lisp code `read'-able in the `eval-package'.
`(.input-area
:background-color ,theme:secondary
:width "99%"
:padding "0.5rem"
:margin "1rem")
:padding "0.5rem")
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't believe this change merits a mention in the changelog.

@jmercouris jmercouris mentioned this pull request Dec 21, 2023
@jmercouris
Copy link
Member Author

Obligatory screenshot:

Screenshot_20231221_151554

message.lisp: Convert to using new top-menu.

buffer-listing.lisp: adapt to use the new menu.

history.lisp: use menu bar.

download.lisp: Add menu to download page.

bookmark.lisp: add menu to bookmark menu.

macro-edit.lisp: Add render menu to edit macro.

plaintext-editor.lisp: Add a menu for the editor.
This prevents scrolling in the X direction.
@aadcg aadcg merged commit aa877fd into master Dec 22, 2023
2 checks passed
@aadcg aadcg deleted the mode-menu-bar branch December 22, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

built in text-editor
2 participants