Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Use project-switch-commands for project-el backend of dashboard widget #265

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

Closed
wants to merge 2 commits into from
Closed

Use project-switch-commands for project-el backend of dashboard widget #265

wants to merge 2 commits into from

Conversation

rudi
Copy link
Contributor

@rudi rudi commented Dec 3, 2020

The project package has a command project-switch-project which lets the user choose first a project and then a command from a customizable list; e.g. find a file, open magit, start a shell, etc.

This pull request implements presenting the same list of commands from the project widget when dashboard-projects-backend is set to project-el, instead of always prompting for a file to open.

rudi added 2 commits December 3, 2020 10:17
- The project package built into Emacs 27 is too old; refer to Gnu
  Elpa’s version instead.
- Instead of always prompting which file to load, let the user choose
  from the list of actions that `project-switch-project` uses.
@rudi rudi changed the title Have the projects widget use commands from project-switch-commands Use project-switch-commands for project-el backend of dashboard widget Dec 3, 2020
@jcs090218 jcs090218 self-assigned this Dec 3, 2020
@jcs090218
Copy link
Member

Sorry, I think there are conflict after this PR #232. Can you resolve it for us? Thank you!

@rudi
Copy link
Contributor Author

rudi commented Dec 9, 2020

I think after #232 this pull request might not be necessary anymore since the user can modify the behavior themselves. I created #271 that documents how to use dashboard-projectile-switch-function and fix the customize widget of that variable.

@jcs090218
Copy link
Member

Thanks for explaining it! I thought this PR was different to #232. 😅

Thanks for the contribution! I appreciated! ;)

@jcs090218 jcs090218 closed this Dec 9, 2020
@rudi
Copy link
Contributor Author

rudi commented Dec 9, 2020

It does something different than #232, but just replaces one hard-coded behavior with another one. #232 enables the user to choose the behavior themselves, so I think having the two project backends behave in the same way by default (i.e., prompt which file to open) is better overall for the project.

Thanks for working on dashboard! It's my default buffer for each new tab, and very useful overall.

@rudi rudi deleted the project-el-switch-commands branch December 9, 2020 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants