Tratamemento para _hover_ e _completion_.#73
Tratamemento para _hover_ e _completion_.#73brodao-zz wants to merge 3 commits intoAlencarGabriel:masterfrom
Conversation
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.
|
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. |
|
@brodao ? |
|
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
<#73 (comment)>,
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 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 |
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.