Skip to content

Conversation

@brodao-zz
Copy link

Foi adicionado tratamento para verificação de instalação do TDS-VSCode 1.4.0, para que esta extensão não sobrescreva as funcionalidades de hover e completion que serão liberadas em breve.

Também atualizei as dependências de extensões no package.json para as mais atuais, porém se quiser pode ignorar que não afetará o tratamento que adicionei.

Agradeço se incorporar na sua distribuição.

PS: Parabéns pela extensão ficou muito bem feita.

Atualização das versões de pacotes dependentes.
Adicionado tratamento para verificação de instalação do TDS-VSCode 1.4.0, para que esta extensão não sobrescreva as funcionalidades de _hover_ e _completion_ que serão liberadas em breve.
@AlencarGabriel
Copy link
Owner

Olá @brodao , obrigado pelo seu PR.

Estive avaliando aqui, e pelo que entendi o registerHoverProvider é o que está conflitando com a TDS. Você chegou a investigar o real problema? Pois pelo que sei o VSCode permite multiplos providers, e esse é um dos recursos mais utilizados da minha extensão, talvez seja o caso tratar o problema, e não retirar o recurso.

@AlencarGabriel
Copy link
Owner

@brodao ?

@brodao-zz
Copy link
Author

brodao-zz commented Sep 22, 2022 via email

@AlencarGabriel
Copy link
Owner

Opa @brodao , obrigado pelos elogios, e agradeço colocar a extensão nas recomendações da TDS, será um prazer!

Sobre a feature, entendi melhor a situação agora. Quando há mais de um hover provider o VSCode irá mostrar as duas informações, uma abaixo da outra, e é rankeado por quem resolveu de forma decrescente.

O hover que eu implementei é totalmente baseado no ProtheusDoc das funções/métodos/classes, e provavelmente o de vocês será mais "confiável", visto olhar - acredito eu - a estrutura do código, e não do ProtheusDoc (me corrija se eu estiver errado).

Se for o caso, a gente pode reestruturar o hover desse lado, para quando o usuário estiver usando a TDS, mostrar apenas os dados que vocês não tratarem, ex.: @history

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.

3 participants