yats.vim icon indicating copy to clipboard operation
yats.vim copied to clipboard

Improvment list

Open HerringtonDarkholme opened this issue 7 years ago • 16 comments

  • [ ] Add typescriptFunctionCall highlight group #13
  • [x] Add travis ci
  • [x] Improve performance in Vue file
  • [x] Simplify typescriptObjectLabel regex
  • [x] Reenable typescriptProp's transparent group
  • [ ] Set default value in argument list as transparent
  • [x] fix return {obj: true} not highlighted in vue
  • [x] methods overloading in class body
  • [x] cleanup typescriptValue and typescriptExpression
  • [x] update data directory to sync with yajs
  • [x] simplify class/interface body highlight
  • [x] unify ambient class and normal class
  • [x] change type highlight color, reference -> Type, predefined -> Literal
  • [ ] proper support for destructruing pattern
  • [x] tsx, yup, tsx 😀😀😀😀😀

HerringtonDarkholme avatar Jan 18 '18 07:01 HerringtonDarkholme

Super excited to see TSX on this list! Really looking forward to trying it. :)

BlakeWilliams avatar Feb 19 '18 20:02 BlakeWilliams

I have just subscribed to this issues.

Would it be possible, to leave a message here whenever a box gets ticked? That way people can subscribe to this and just get notified. I would love to help but unfortunately my vim scripts are really basic.

FabioAntunes avatar Mar 19 '18 15:03 FabioAntunes

@HerringtonDarkholme, I'd love to help with TSX, would you be open to sharing any thoughts you have on how you were planning on including the XML syntax in TSX? I don't want to start work on something that wouldn't end up making it into yats.vim if I can avoid it.

pcflmb avatar Mar 20 '18 20:03 pcflmb

this is amazing.. tsx support would be the illest. yeah anything your user base might be able to help with on this?

megalithic avatar May 15 '18 17:05 megalithic

@HerringtonDarkholme is there anything we can help with? do you have a clear direction on tsx support? i'd love to contribute as this issue: https://github.com/HerringtonDarkholme/yats.vim/issues/47 affects me daily, so i'd love to contribute if you have some direction to give. :)

I'm selfishly going to /cc @mhartington to see if he has any insight/thoughts around this (might not affect him all that much since i think he's mainly in angular land)

megalithic avatar May 17 '18 18:05 megalithic

Angular and JSX/TSX these days 😄

Happy to lend a hand where I can

mhartington avatar May 17 '18 20:05 mhartington

@mhartington have you had issues with syntax rendering related to to this previously closed issue?

I reckon I'll give OceanicNext a go and see if it renders everything right.

EDIT: still the same :( -- https://cl.ly/2Y1D3s3H0Z2B

megalithic avatar May 17 '18 22:05 megalithic

I have already pushed a preliminary support for JSX. It still needs polish but you can try it out today.

But I'm too busy with my day job. I really don't have any time to devote in OSS...

HerringtonDarkholme avatar May 18 '18 02:05 HerringtonDarkholme

@HerringtonDarkholme Where is the preliminary support for jsx? And how can we try it out? I don't see any branches besides master. Is it on master?

agarrharr avatar May 18 '18 13:05 agarrharr

JSX support is on master. Currently it supports basic <tag> and fragment <>.

HerringtonDarkholme avatar May 18 '18 17:05 HerringtonDarkholme

Current progress:

screenshot from 2018-05-20 21-47-56

I have got fragment highlighted. Also custom component and intrinsic tag have different highlighting!

HerringtonDarkholme avatar May 20 '18 13:05 HerringtonDarkholme

I don't use JSX personally. Would you like recommend a good library to test highlighting?

I've searched GitHub by query: typescript + react. https://github.com/ant-design/ant-design seems like a high-starred project.

HerringtonDarkholme avatar May 20 '18 13:05 HerringtonDarkholme

Nice! Really excited to start using this soon.

I have a few snippets of JSX that I know other syntaxes have trouble with. Mostly around JSX inside of {} and template strings.

render() {
  return (
    <React.Fragment>
      {this.state.thing ? <p>Hello <strong data-role={`${this.props.prefix}-name`}>world</strong></p> : <h1>Hey!</h1>}
      <h1 className={`${this.props.baz}-class`}>wow</h1>
    </React.Fragment>
  )
}

I want to say those are what trip up a different JSX syntax for TS that I've used.

BlakeWilliams avatar May 20 '18 14:05 BlakeWilliams

@BlakeWilliams Thanks for report! I have fixed it!

I have opened a new issue dedicated to JSX. Please file JSX problem there so I can focus JSX stuff :)

HerringtonDarkholme avatar May 20 '18 15:05 HerringtonDarkholme

Is there still no progress on typescriptFunctionCall?

ghost avatar Oct 01 '19 10:10 ghost

I'd like to make a PR for it but I don't even know where to start

ghost avatar Oct 01 '19 13:10 ghost