vscord icon indicating copy to clipboard operation
vscord copied to clipboard

Rework the config system.

Open xhayper opened this issue 2 years ago • 11 comments

Currently, the system is very messy due to we :cough: having too much customization option, we need to design a better system to reduce the complexity and confusion.

xhayper avatar Nov 27 '22 11:11 xhayper

I agree

leonardssh avatar Nov 27 '22 11:11 leonardssh

~~and i though the first rework makes it less complicate... it doesn't...~~

xhayper avatar Nov 27 '22 11:11 xhayper

@nick22985 calling you in because you are very active contributor.

xhayper avatar Nov 27 '22 11:11 xhayper

I agree lol, time to make it ~~more~~ less messy xD

nick22985 avatar Nov 27 '22 11:11 nick22985

I was thinking of a small scrippting lang so we can merge tons of stuff and provide all customization.

xhayper avatar Nov 27 '22 11:11 xhayper

oooo sounds cool

nick22985 avatar Nov 27 '22 11:11 nick22985

but the problem is, me, you and leonard don't know how to make AST / parser / lexer xD.

xhayper avatar Nov 27 '22 11:11 xhayper

if in workspace then
   if git is active then
      // TODO: decide how to provide variable
      // maybe {{}} ? or just treat it like js, var
      // global.gitLink
      set button.one.url to 
      set button.one.text to "View repository"
   end
end

xhayper avatar Nov 27 '22 11:11 xhayper

but the problem is, me, you and leonard don't know how to make AST / parser / lexer xD.

yea

nick22985 avatar Nov 27 '22 11:11 nick22985

It's a really hard thing to do.

xhayper avatar Nov 27 '22 12:11 xhayper

Yea it doesn't seem very easy. I wouldn't even know where we would put the config file.

We probably could cut down a bit on the config options by making some of their tables with enums (if enums work I haven't really tested it) Something like this(https://gyazo.com/14fdbd556a6ae8981d80334d9936fced) for each section that has state makes an object which is usable and can easily be changed in the extension without a massive refactor https://gyazo.com/3aad601a80d78c079db63b97e325363a

nick22985 avatar Nov 29 '22 03:11 nick22985