-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Support system/user default clone editors #22378
Conversation
This should be option to allow select multiple. My use case: on work laptop I use VSCode but on personal laptop I use VSCodium |
I will add multiple editors support. |
Looks nice. I think it should also be possible to use a custom editor in case a User uses some odd editor that is not in the list. You could add a radio button "Use editor from list" which is the current list of your PR and "Use custom editor" where the User can Input a Name, URL and optional a Icon. |
That should be next PR for customized editors. |
Honestly imo this doesn't fix the core issue, it just moves it around. I think Gitea should allow instance admins to configure whichever they want (and still allow per-user custom anyways) |
That's what's the next step to do. |
Codecov Report
@@ Coverage Diff @@
## main #22378 +/- ##
=========================================
+ Coverage 0 48.17% +48.17%
=========================================
Files 0 1045 +1045
Lines 0 142503 +142503
=========================================
+ Hits 0 68645 +68645
- Misses 0 65680 +65680
- Partials 0 8178 +8178
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Co-authored-by: Yarden Shoham <hrsi88@gmail.com> Co-authored-by: Lauris BH <lauris@nix.lv>
@yardenshoham @lafriks Both done. |
Referencevivaldi_pKV6RbHvOb.mp4 |
How could we know the users' editors within the webbrowser? |
As I commented also in other PR, I don't see from current UI perspective much of the problem if in menu I have one option (Open in xxx editor) or multiple, so I don't see how that would make it much easier to use it if default editor would be selected per browser? |
we don't need to, we only need to know all their preferences (which per will enter) and which one to pick on that particular webbrowser (which again, the user will select perself ofcourse). After that, that editor will be shown in the repos etc as default. |
Needs a rebase |
I guess this PR is not mature yet (the implement doesn't seem maintainable for future). |
Maybe non-ui config would be simpler to implement, e.g. #21121 (comment). Downside is only admin can change it. |
-> Customizable "Open with" applications for repository clone #29320 |
Automatically locked because of our CONTRIBUTING guidelines |
Replace #21123
Close #21121
Close #22149
This PR introduces a global configuration named
dev.default_editors
and a user level configuration nameddev.default_editors
. Login users will see the menu items from him own configuration and anonymous user will see the menus from global configurations.Currently, it embeded 3 editors,
vscode
,vscodium
andIDEA
. More influent editors and IDE could be added.TODO:
Admin panel configuration
User setting configuration
UI in the repository home page