Tratamemento para _hover_ e _completion_.

brodao-zz commented 2 years ago

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.

AlencarGabriel commented 2 years ago

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 commented 1 year ago

@brodao ?

brodao-zz commented 1 year ago

Ola Alencar,

Desculpe a demora em responder. Sem dúvida que sua extensão é excelente (me passou a perna, tava pensando em fazê-la por conta própria ;) ). A cerca de um mês, me passaram essa incumbência de forma oficial para implementar no LS, do lado binário e na nossa extensão o hover, que será mais parrudo e o seu hover, ou sobrepõe a nossa funcionalidade ou causa duplicidade de informações. Como pode ver no PR, coloquei uma versão futura do VS-Code para suprimir a sua.

Os demais recursos ainda irão continuar sendo de extrema importância (e me poupará muito trabalho) e inclusive iremos colocar a sua como recomendação pelos demais recursos que oferece.

Por enquanto, deixa a PR em espera (ou mesma recusa) e quando eu terminar a implementação, vou estudar uma forma de manter os dois hover. Talvez coloque uma opção de trabalhar com as duas ou somente com uma delas.

À disposição.

[:>} Alan Cândido ><>

Especialista de Pesquisa e DesenvolvimentoExpert Research and Development .[====].[====].(----).(----)._/.[_].]@|--Y_ _oo_oooooo_ooo_o__Ooo_\ Quentinha para quem não tem um teto https://www.vakinha.com.br/vaquinha/quentinha-para-quem-nao-tem-um-teto

Em qua., 21 de set. de 2022 às 15:53, Gabriel Alencar < @.***> escreveu:

@brodao https://github.com/brodao ?

— Reply to this email directly, view it on GitHub https://github.com/AlencarGabriel/ProtheusDoc-VsCode/pull/73#issuecomment-1254100953, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHH5GRAEN7BLZWB533YWHTV7NKTLANCNFSM6AAAAAAQGIVEGY . You are receiving this because you were mentioned.Message ID: @.***>

AlencarGabriel commented 1 year ago

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

AlencarGabriel / ProtheusDoc-VsCode

Tratamemento para _hover_ e _completion_. #73