ProtheusDoc-VsCode
ProtheusDoc-VsCode copied to clipboard
AlencarGabriel
Tratamemento para _hover_ e _completion_.
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.
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.
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_
o__o___o__o___o__o___o__o___o_o___o_o___O___oo
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: @.***>
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