Skip to content
This repository was archived by the owner on Mar 19, 2024. It is now read-only.

In-client IDE #7

Open
CmorDusMor opened this issue Jul 14, 2023 · 9 comments
Open

In-client IDE #7

CmorDusMor opened this issue Jul 14, 2023 · 9 comments
Assignees
Labels
beta-essential Essential for beta release Essential Crucial feature we can't publish without

Comments

@CmorDusMor
Copy link

"We need to figure out how to quickly set up Web VScode with a classpath that allows for syntax highlighting, for module development (since modules will be made by public, random nons)
It does not work because it is not implemented... at all.
It would work by doing something with vscode in the web, or installing some kind of editor."
-Saturn5Vfive

Frankly I'm not quite sure what this is about, or if it's very solid of an idea, or what it actually is that's being explained here. I'm giving this to saturn5Vfive to do until this is more concrete.

@CmorDusMor CmorDusMor added enhancement New feature or request question Further information is requested labels Jul 14, 2023
@CmorDusMor
Copy link
Author

@Saturn5Vfive please explain more about this idea if you want to give it to someone else.

@HyperCodec
Copy link
Collaborator

"We need to figure out how to quickly set up Web VScode with a classpath that allows for syntax highlighting, for module development (since modules will be made by public, random nons) It does not work because it is not implemented... at all. It would work by doing something with vscode in the web, or installing some kind of editor." -Saturn5Vfive

Frankly I'm not quite sure what this is about, or if it's very solid of an idea, or what it actually is that's being explained here. I'm giving this to saturn5Vfive to do until this is more concrete.

Bro PrismJS exists

@HyperCodec
Copy link
Collaborator

@CmorDusMor @Saturn5Vfive why web IDE? Can't people just use Intellij to create their modules and then upload the files?

@CmorDusMor CmorDusMor assigned stdcout1 and unassigned 0x4D2D and stdcout1 Jul 16, 2023
@stdcout1
Copy link
Contributor

I think we can just use https://codemirror.net/ and wrap it in a container that has options to switch tabs. no need to reinvent the wheel.

@HyperCodec
Copy link
Collaborator

I think we can just use https://codemirror.net/ and wrap it in a container that has options to switch tabs. no need to reinvent the wheel.

That's what I was thinking too. I could make a quick app with Tauri that's also autoinstalled, then the mod can start it with some command args for file and such

@HyperCodec
Copy link
Collaborator

Merged #34, should make it automatically update the ide files in this repo

@HyperCodec HyperCodec linked a pull request Aug 5, 2023 that will close this issue
@HyperCodec HyperCodec removed a link to a pull request Aug 5, 2023
@HyperCodec HyperCodec removed the question Further information is requested label Aug 5, 2023
@CmorDusMor
Copy link
Author

@0x4D2D @stdcout1 sick job getting this to work!
Lefty and I agree on a few things though:

  1. you should be able to press "tab" to complete the tabcomplete (instead of having to manually click it)

  2. you should be able to see more than JUST the top method, ideally the top 4-7, and ideally you should be able to scroll wheel up and down to see all matching methods

  3. method autocomplete needs to be sorted ALPHABETICALLY (NOT by length, by length is really annoying and stupid.) we're both screaming in agony about this.

  4. the IDE location just says "placeholder, please implement when actual" https://cdn.discordapp.com/attachments/1068605392948564110/1139491406159810601/image.png this needs to be ACTUALLY implemented to correctify this feature, obviously

  5. the editor DESPERATELY NEEDS copy+paste support (ctrl+c, ctrl+v)

  6. you need to be able to highlight text by clicking and dragging (currently only double/triple-click works to highlight, and ctrl+a)

  7. i don't know how easy or even possible this would be to fix, but notice the following:

public Class main {
MinecraftClient.getInstance().[all the autocomplete stuff works here]
MinecraftClient mc = MinecraftClient.getInstance();
mc.[nothing shows up for autocomplete here]
}

@CmorDusMor CmorDusMor added Essential Crucial feature we can't publish without and removed enhancement New feature or request labels Aug 11, 2023
@CmorDusMor CmorDusMor changed the title In-client IDE...? In-client IDE Aug 11, 2023
@ghost ghost added the beta-essential Essential for beta release label Aug 31, 2023
@HyperCodec
Copy link
Collaborator

very close to finishing i believe

just gotta get the ide code to link with the lsp

@0x4D2D
Copy link
Collaborator

0x4D2D commented Sep 20, 2023

so true

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
beta-essential Essential for beta release Essential Crucial feature we can't publish without
Projects
None yet
Development

No branches or pull requests

4 participants